FIX: Temporary fix to avoid exceptions being thrown by InputControlPathEditor editor inside UITK IMGUIContainer when deleting binding (#1789)

* Added try-catch block to stop exceptions from being thrown

* Added Changelog entry

---------

Co-authored-by: Håkan Sidenvall <[email protected]>
Co-authored-by: James McGill <[email protected]>

Author: 3 people

Packages/com.unity.inputsystem/CHANGELOG.md

@@ -43,9 +43,9 @@ however, it has to be formatted properly to pass verification tests.
 - Fix for BindingSyntax `WithInteraction()` which was incorrectly using processors.
 - Fix for UITK Input Action Editor binding 'Listen' button which wasn't working in the case for Control Type 'Any'.
 - Fixed issue of visual elements being null during editing project-wide actions in project settings which prompted console errors.
+- Fixed case ISX-1436 (UI TK Input Action Asset Editor - Error deleting Bindings with DeleteKey on Windows).
 - Fixed issue with UI Toolkit based Input Action Editor not restoring it's selected items after Domain Reload.
 
-
 ## [1.8.0-pre.1] - 2023-09-04
 
 ### Added

Packages/com.unity.inputsystem/InputSystem/Editor/ControlPicker/InputControlPathEditor.cs

@@ -109,7 +109,19 @@ public void OnGUI(Rect rect, GUIContent label = null, SerializedProperty propert
             editButtonRect.width = 20;
             editButtonRect.height = 15;
 
-            var path = serializedProperty.stringValue;
+            var path = String.Empty;
+            try
+            {
+                path = serializedProperty.stringValue;
+            }
+            catch
+            {
+                // This try-catch block is a temporary fix for ISX-1436
+                // The plan is to convert InputControlPathEditor entirely to UITK and therefore this fix will
+                // no longer be required.
+                return;
+            }
+
             ////TODO: this should be cached; generates needless GC churn
             var displayName = InputControlPath.ToHumanReadableString(path);