From b4eb89e468119e7fbedafb1a2f69c777bd9d50e4 Mon Sep 17 00:00:00 2001 From: Ameer Armaly Date: Wed, 20 May 2020 13:54:23 -0700 Subject: [PATCH] MultiFingerMultiTap: reset the expiration clock when fingers go up. We were resetting the expiration clock every time a finger went down, but were not resetting it when fingers went up. As a result the user had a super narrow window to get all the fingers up before the gesture canceled. Bug: 156880174 Test: atest GestureManifoldTest Change-Id: I81dd9cd67729cfae38952d44295c3340c5426028 Change-Id: I1b6323ac413bcbd46fd9292fbb4cf7ebdf2b6246 --- .../server/accessibility/gestures/MultiFingerMultiTap.java | 1 + 1 file changed, 1 insertion(+) diff --git a/services/accessibility/java/com/android/server/accessibility/gestures/MultiFingerMultiTap.java b/services/accessibility/java/com/android/server/accessibility/gestures/MultiFingerMultiTap.java index e5340f10dc4c0..642a841257bf7 100644 --- a/services/accessibility/java/com/android/server/accessibility/gestures/MultiFingerMultiTap.java +++ b/services/accessibility/java/com/android/server/accessibility/gestures/MultiFingerMultiTap.java @@ -197,6 +197,7 @@ class MultiFingerMultiTap extends GestureMatcher { if (getState() == STATE_GESTURE_STARTED || getState() == STATE_CLEAR) { // Needs more fingers lifted within the tap timeout // after reaching the target number of fingers are down. + cancelAfterTapTimeout(event, rawEvent, policyFlags); } else { cancelGesture(event, rawEvent, policyFlags); }