diff options
author | 2025-01-13 23:50:35 +0000 | |
---|---|---|
committer | 2025-01-14 20:42:16 +0000 | |
commit | 3192276a439484d1e7da50aef7ca1a180f78dcd6 (patch) | |
tree | f6421ab233d424565a51657b5de90307422840c6 /services/accessibility/accessibility.aconfig | |
parent | 28cef9b76c59eb2e8dd4b2614a70217608b978fb (diff) |
Cleanup EventDispatcher#sendMotionEvent callers to provide raw event.
A few releases ago this codebase started tracking the raw (unmodified)
and regular (possibly modified) event separately, but some older
sendMotionEvent callers were not updated to provide the correct
raw event.
This should also help prevent a race condition NPE in
TouchExplorer#sendHoverExitAndTouchExplorationGestureEndIfNeeded:
That method was using the last received *regular* event instead of the
raw event, and this regular event was already cleared from a previous
call to clear() so EventDispatcher#sendMotionEvent was unable to
access the event's attributes. (See linked bug)
Bug: 385812366
Test: atest TouchExplorerTest
Test: atest FrameworksServicesTests:com.android.server.accessibility
Test: Use TalkBack touch exploration, observe no behavior changes
Flag: com.android.server.accessibility.event_dispatcher_raw_event
Change-Id: Ie60f8174de3b12da01987f9bad59669f45bbdfdf
Diffstat (limited to 'services/accessibility/accessibility.aconfig')
-rw-r--r-- | services/accessibility/accessibility.aconfig | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/services/accessibility/accessibility.aconfig b/services/accessibility/accessibility.aconfig index 7f0bf0375b4a..722255404dd1 100644 --- a/services/accessibility/accessibility.aconfig +++ b/services/accessibility/accessibility.aconfig @@ -145,6 +145,16 @@ flag { } flag { + name: "event_dispatcher_raw_event" + namespace: "accessibility" + description: "Fixes EventDispatcher#sendMotionEvent callers to properly provide raw event" + bug: "385812366" + metadata { + purpose: PURPOSE_BUGFIX + } +} + +flag { name: "fix_drag_pointer_when_ending_drag" namespace: "accessibility" description: "Send the correct pointer id when transitioning from dragging to delegating states." |