Add improvements based on review

Author: jfreire-unity

Assets/Tests/InputSystem/CoreTests_Actions.cs

@@ -687,6 +687,7 @@ public void Actions_DoNotGetTriggeredByOutOfFocusEventInEditor(InputSettings.Bac
             Assert.That(actions, Has.Length.EqualTo(backgroundBehavior == InputSettings.BackgroundBehavior.IgnoreFocus ? 2 : 1));
             Assert.That(actions[0].phase, Is.EqualTo(InputActionPhase.Performed));
             Vector2Control control = (Vector2Control)actions[0].control;
+            // Make sure the value is from the event after focus was regained.
             Assert.That(control.value, Is.EqualTo(new Vector2(1.0f, 2.0f)).Using(Vector2EqualityComparer.Instance));
         }
     }

Packages/com.unity.inputsystem/CHANGELOG.md

@@ -28,7 +28,7 @@ however, it has to be formatted properly to pass verification tests.
 - Fixed the compilation warnings when used with Unity 6.4 (ISX-2349).
 - Fixed an issue where `InputSystemUIInputModule.localMultiPlayerRoot` could not be set to `null` when using `MultiplayerEventSystem`. [ISXB-1610](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1610)
 - Fixed an issue in `Keyboard` where the sub-script operator would return a `null` key control for the deprecated key `Key.IMESelected`. Now, an aliased `KeyControl`mapping to the IMESelected bit is returned for compability reasons. It is still strongly advised to not rely on this key since `IMESelected` bit isn't strictly a key and will be removed from the `Key` enumeration type in a future major revision. [ISXB-1541](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1541).
-- An issue where a UITK MouseEvent was triggered when changing from Scene View to Game View in the Editor has been fixed.  [ISXB-1671](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1671)
+- An issue where a UITK MouseEvent was triggered when changing from Scene View to Game View in the Editor has been fixed. [ISXB-1671](https://issuetracker.unity3d.com/product/unity/issues/guid/ISXB-1671)
 
 ## [1.14.2] - 2025-08-05
 

Packages/com.unity.inputsystem/InputSystem/InputManager.cs

@@ -3210,7 +3210,9 @@ private unsafe void OnUpdate(InputUpdateType updateType, ref InputEventBuffer ev
             var shouldFlushEventBuffer = ShouldFlushEventBuffer();
             // When we exit early, we may or may not flush the event buffer. It depends if we want to process events
             // later once this method is called.
-            var shouldExitEarly = ShouldExitEarlyFromEventProcessing(eventBuffer, shouldFlushEventBuffer, updateType);
+            var shouldExitEarly =
+                eventBuffer.eventCount == 0 || shouldFlushEventBuffer || ShouldExitEarlyFromEventProcessing(updateType);
+
 
 #if UNITY_EDITOR
             var dropStatusEvents = ShouldDropStatusEvents(eventBuffer, ref shouldExitEarly);
@@ -3692,16 +3694,8 @@ private bool ShouldFlushEventBuffer()
         /// <param name="canFlushBuffer">Whether the buffer can be flushed</param>
         /// <param name="updateType">The current update type</param>
         /// <returns>True if we should exit early, false otherwise.</returns>
-        private bool ShouldExitEarlyFromEventProcessing(InputEventBuffer eventBuffer, bool canFlushBuffer, InputUpdateType updateType)
+        private bool ShouldExitEarlyFromEventProcessing(InputUpdateType updateType)
         {
-            // Early out if there are no events to process
-            if (eventBuffer.eventCount == 0)
-                return true;
-
-            // Early out if we can flush the buffer
-            if (canFlushBuffer)
-                return true;
-
 #if UNITY_EDITOR
             // Check various PlayMode specific early exit conditions
             if (ShouldExitEarlyInEditor(updateType))
@@ -3768,7 +3762,7 @@ private bool ShouldDropStatusEvents(InputEventBuffer eventBuffer, ref bool canEa
         /// <summary>
         /// Determines if an event should be discarded based on timing or focus state.
         /// </summary>
-        /// <param name="eventType">The type of the current event</param>
+        /// <param name="eventType">The type of event</param>
         /// <param name="eventTime">The internal time of the current event</param>
         /// <param name="updateType">The current update type</param>
         /// <returns>True if the event should be discarded, false otherwise.</returns>
@@ -3811,6 +3805,7 @@ private bool ShouldDiscardEditModeTransitionEvent(FourCC eventType, double event
         /// </summary>
         private bool ShouldDiscardOutOfFocusEvent(double eventTime)
         {
+            // If we care about focus, check if the event occurred while out of focus based on its timestamp.
             if (gameHasFocus && m_Settings.backgroundBehavior != InputSettings.BackgroundBehavior.IgnoreFocus)
                 return m_DiscardOutOfFocusEvents && eventTime < m_FocusRegainedTime;
             return false;