diff options
author | 2025-02-21 18:27:14 +0000 | |
---|---|---|
committer | 2025-03-04 18:35:07 +0000 | |
commit | b57366c5021e018d8eeae62853128cf5fdb4f2dc (patch) | |
tree | c1de0406f9f71373a3513af690b27ab3ce8c97bb /services/accessibility | |
parent | 9daca6f94ed29b4547c63206cbe2f334e400d250 (diff) |
(2/n) Remove usage of isKeyGestureSupported() from KeyGestureController
isKeyGestureSupported is synchronous API that makes system server
dependent on handler and can cause ANR if handlers misbehave.
It was added as a stop gap for multi-key gestures that require
some SysUI info like keyguard state, display state, etc, to decide
if the key needs to be consumed or not at the start of the gesture.
With this refactor:
- WM policy added an API to check isKeyguardLocked() and we will
use that for checking keyguard status
- Move A11yShortcutController to KGC to circumvent the need of the
isShortcutAvailable() API
- Since the only difference with TV and non TV event handling was
whether keyguard showing blocks the shortcut or not, we no longer
need to have separate handler logic for TV (we can cover it in
precondition check).
In subsequent CLs:
- Will remove usages of isKeyGestureSupported() API everywhere
Test: atest KeygestureControllerTests
Bug: 358569822
Bug: 383602794
Flag: EXEMPT refactor
Change-Id: Ib5d5c727fabb28623efa6247e817cf0c643a949c
Diffstat (limited to 'services/accessibility')
0 files changed, 0 insertions, 0 deletions