Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix screen turns off once triggering long press power behavior" into udc-qpr-dev

Browse Source
This commit is contained in:
Treehugger Robot
2023-07-05 03:17:11 +00:00
committed by Android (Google) Code Review
parent 5e49928b27 7155f5e59e
commit 01a9c09f67
2 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
services/core/java/com/android/server/policy/SingleKeyGestureDetector.java
View File

@@ -314,7 +314,9 @@ public final class SingleKeyGestureDetector {
if (eventTime < mLastDownTime + mActiveRule.getVeryLongPressTimeoutMs()) {
mHandler.removeMessages(MSG_KEY_VERY_LONG_PRESS);
} else {
mHandledByLongPress = mActiveRule.supportVeryLongPress();
// If long press or very long press (~3.5s) had been handled, we should skip the
// short press behavior.
mHandledByLongPress |= mActiveRule.supportVeryLongPress();
}
}

12
services/tests/wmtests/src/com/android/server/policy/SingleKeyGestureTests.java
View File

@@ -316,4 +316,16 @@ public class SingleKeyGestureTests {
pressKey(KEYCODE_POWER, 0 /* pressTime */);
assertTrue(mShortPressed.await(mWaitTimeout, TimeUnit.MILLISECONDS));
}
// Verify short press should not be triggered if no very long press behavior defined but the
// press time exceeded the very long press timeout.
@Test
public void testTimeoutExceedVeryLongPress() throws InterruptedException {
mVeryLongPressOnPowerBehavior = false;
pressKey(KEYCODE_POWER, mVeryLongPressTime + 50);
assertTrue(mLongPressed.await(mWaitTimeout, TimeUnit.MILLISECONDS));
assertEquals(mVeryLongPressed.getCount(), 1);
assertEquals(mShortPressed.getCount(), 1);
}
}