Also expose current focus state for HandsProvider

Author: keveleigh

org.mixedrealitytoolkit.input/MRTK.Input.asmdef

@@ -50,6 +50,11 @@
             "name": "com.unity.xr.openxr",
             "expression": "",
             "define": "UNITY_OPENXR_PRESENT"
+        },
+        {
+            "name": "com.qualcomm.snapdragon.spaces",
+            "expression": "",
+            "define": "SNAPDRAGON_SPACES_PRESENT"
         }
     ],
     "noEngineReferences": false

org.mixedrealitytoolkit.input/Subsystems/Hands/HandsProvider.cs

@@ -61,13 +61,11 @@ public override bool TryGetEntireHand(XRNode handNode, out IReadOnlyList<HandJoi
         {
             Debug.Assert(handNode == XRNode.LeftHand || handNode == XRNode.RightHand, "Non-hand XRNode used in TryGetEntireHand query.");
 
-#if !UNITY_EDITOR
-            if (!Application.isFocused)
+            if (!MRTKInputFocusManager.HasFocus)
             {
                 jointPoses = Array.Empty<HandJointPose>();
                 return false;
             }
-#endif // !UNITY_EDITOR
 
             return hands[handNode].TryGetEntireHand(out jointPoses);
         }
@@ -77,13 +75,11 @@ public override bool TryGetJoint(TrackedHandJoint joint, XRNode handNode, out Ha
         {
             Debug.Assert(handNode == XRNode.LeftHand || handNode == XRNode.RightHand, "Non-hand XRNode used in TryGetJoint query.");
 
-#if !UNITY_EDITOR
-            if (!Application.isFocused)
+            if (!MRTKInputFocusManager.HasFocus)
             {
                 jointPose = default;
                 return false;
             }
-#endif // !UNITY_EDITOR
 
             return hands[handNode].TryGetJoint(joint, out jointPose);
         }

org.mixedrealitytoolkit.input/Utilities/MRTKInputFocusManager.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Mixed Reality Toolkit Contributors
 // Licensed under the BSD 3-Clause
 
+using System.Collections.Generic;
 using UnityEngine;
 using UnityEngine.Events;
 using UnityEngine.InputSystem;
@@ -20,6 +21,18 @@ public sealed class MRTKInputFocusManager : MonoBehaviour
         /// </summary>
         public static UnityEvent<bool> OnXrSessionFocus { get; } = new UnityEvent<bool>();
 
+        /// <summary>
+        /// Whether the current XrSession has focus or not.
+        /// </summary>
+        public static bool HasFocus =>
+#if UNITY_EDITOR
+            true;
+#elif SNAPDRAGON_SPACES_PRESENT
+            lastSessionState == 5;
+#else
+            Application.isFocused;
+#endif
+
         /// <summary>
         /// We want to ensure we're focused for input, as some runtimes continue reporting "tracked" while pose updates are paused.
         /// This is allowed, per-spec, as a "should": "Runtimes should make input actions inactive while the application is unfocused,
@@ -43,15 +56,25 @@ private void OnFocusChange(bool focus)
             }
         }
 
-#if USING_SNAPDRAGON_SPACES_SDK
+#if SNAPDRAGON_SPACES_PRESENT
+        private static readonly List<Qualcomm.Snapdragon.Spaces.SpacesOpenXRFeature> featureList = new();
+        private static int lastSessionState = -1;
         private Qualcomm.Snapdragon.Spaces.SpacesOpenXRFeature spacesOpenXRFeature = null;
-        private int lastSessionState = -1;
 
         private void Update()
         {
             if (spacesOpenXRFeature == null)
             {
-                spacesOpenXRFeature = UnityEngine.XR.OpenXR.OpenXRSettings.Instance.GetFeature<Qualcomm.Snapdragon.Spaces.SpacesOpenXRFeature>();
+                int count = UnityEngine.XR.OpenXR.OpenXRSettings.Instance.GetFeatures(featureList);
+                for (int i = 0; i < count; i++)
+                {
+                    Qualcomm.Snapdragon.Spaces.SpacesOpenXRFeature feature = featureList[i];
+                    if (feature != null && feature.enabled)
+                    {
+                        spacesOpenXRFeature = feature;
+                        break;
+                    }
+                }
             }
 
             // XrSessionState maps better to this behavior than OnApplicationFocus but isn't