FIX: remove unneeded check that triggered NotImplementedException for Usage bindings (#1625)

Author: jamesmcgill

Packages/com.unity.inputsystem/CHANGELOG.md

@@ -24,6 +24,7 @@ however, it has to be formatted properly to pass verification tests.
 - Fixed Switch Pro, DualShock 4, DualSense gamepads becoming current on PC/macOS when no controls are changing ([case ISXB-223](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-223))).
 - Fixed an issue that made OnScreenStick unusable when used in conjunction with PlayerInput in Auto-Switch devices mode, or with any code that changes user/device pairing on unsued device activity being detected ([case ISXB-48](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-48)).
 - Fixed issue where input events were being suppressed during interactive action rebinding even when when their controls were excluded ([case ISXB-367](https://issuetracker.unity3d.com/issues/mouse-position-and-mouse-click-input-not-recognized-when-rebinding-is-active)).
+- Removed unneeded check that could trigger a NotImplementedException when binding to a Usage (e.g. Submit) ([case ISXB-373](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-373)).
 - Display a warning instead of throwing a NotImplementedException when loading binding overrides from json when some of the entries have become outdated ([case ISXB-375](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-375)).
 
 ### Actions

Packages/com.unity.inputsystem/InputSystem/Controls/InputControlPath.cs

@@ -901,11 +901,6 @@ private static TControl MatchControlsRecursive<TControl>(InputControl control, s
                     TControl lastMatch;
                     if (path[indexInPath] == '{')
                     {
-                        ////TODO: support scavenging a subhierarchy for usages
-                        if (!ReferenceEquals(control.device, control))
-                            throw new NotImplementedException(
-                                "Matching usages inside subcontrols instead of at device root");
-
                         // Usages are kind of like entry points that can route to anywhere else
                         // on a device's control hierarchy and then we keep going from that re-routed
                         // point.