diff options
| author | 2021-05-18 20:07:17 +0000 | |
|---|---|---|
| committer | 2021-05-18 20:07:17 +0000 | |
| commit | c1c3ab17eadfe950fc9a6cfcc973b03d1e44e8c2 (patch) | |
| tree | 6283413a81db4fca1ac2f2abcc7f892e4018de06 | |
| parent | faf3969c2c05ec1b2e2bdc17d65119ae98a7b8db (diff) | |
| parent | 8e1884245009791e6a28fb40b450c85024724b8b (diff) | |
Merge "Don't ignore gestures on pulsing." into sc-dev
| -rw-r--r-- | packages/SystemUI/src/com/android/systemui/classifier/FalsingCollectorImpl.java | 4 | ||||
| -rw-r--r-- | packages/SystemUI/tests/src/com/android/systemui/classifier/FalsingCollectorImplTest.java | 20 |
2 files changed, 22 insertions, 2 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/classifier/FalsingCollectorImpl.java b/packages/SystemUI/src/com/android/systemui/classifier/FalsingCollectorImpl.java index 0f202b085851..969736910b5e 100644 --- a/packages/SystemUI/src/com/android/systemui/classifier/FalsingCollectorImpl.java +++ b/packages/SystemUI/src/com/android/systemui/classifier/FalsingCollectorImpl.java @@ -257,7 +257,9 @@ class FalsingCollectorImpl implements FalsingCollector { @Override public void onTouchEvent(MotionEvent ev) { - if (!mKeyguardStateController.isShowing() || mStatusBarStateController.isDozing()) { + if (!mKeyguardStateController.isShowing() + || (mStatusBarStateController.isDozing() + && !mStatusBarStateController.isPulsing())) { avoidGesture(); return; } diff --git a/packages/SystemUI/tests/src/com/android/systemui/classifier/FalsingCollectorImplTest.java b/packages/SystemUI/tests/src/com/android/systemui/classifier/FalsingCollectorImplTest.java index bc2444546352..3c41216949c2 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/classifier/FalsingCollectorImplTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/classifier/FalsingCollectorImplTest.java @@ -21,6 +21,7 @@ import static org.mockito.ArgumentMatchers.argThat; import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.never; import static org.mockito.Mockito.reset; +import static org.mockito.Mockito.times; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -185,7 +186,7 @@ public class FalsingCollectorImplTest extends SysuiTestCase { } @Test - public void testAvoidDozing() { + public void testAvoidDozingNotPulsing() { MotionEvent down = MotionEvent.obtain(0, 0, MotionEvent.ACTION_DOWN, 0, 0, 0); MotionEvent up = MotionEvent.obtain(0, 0, MotionEvent.ACTION_UP, 0, 0, 0); @@ -199,4 +200,21 @@ public class FalsingCollectorImplTest extends SysuiTestCase { mFalsingCollector.onTouchEvent(up); verify(mFalsingDataProvider, never()).onMotionEvent(any(MotionEvent.class)); } + + @Test + public void testAvoidDozingButPulsing() { + MotionEvent down = MotionEvent.obtain(0, 0, MotionEvent.ACTION_DOWN, 0, 0, 0); + MotionEvent up = MotionEvent.obtain(0, 0, MotionEvent.ACTION_UP, 0, 0, 0); + + when(mStatusBarStateController.isDozing()).thenReturn(true); + when(mStatusBarStateController.isPulsing()).thenReturn(true); + + // Nothing passed initially + mFalsingCollector.onTouchEvent(down); + verify(mFalsingDataProvider, never()).onMotionEvent(any(MotionEvent.class)); + + // Up event would flushes + mFalsingCollector.onTouchEvent(up); + verify(mFalsingDataProvider, times(2)).onMotionEvent(any(MotionEvent.class)); + } } |