Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f1cda83c authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Add a 5 second timeout to launch animation touch blocking" into udc-d1-dev am: 9aba81b3 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/24479294



Change-Id: I35c1780baa74be0daad15a7125e8879e2d33a86d
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents afd3d0b1 9aba81b3

packages/SystemUI/src/com/android/systemui/shade/NotificationShadeWindowViewController.java

+16 −1
Original line number Diff line number Diff line
@@ -105,7 +105,14 @@ public class NotificationShadeWindowViewController { 
    private boolean mTouchActive;

    private boolean mTouchCancelled;

    private MotionEvent mDownEvent;

    // TODO rename to mLaunchAnimationRunning

    private boolean mExpandAnimationRunning;

    /**

     *  When mExpandAnimationRunning is true and the touch dispatcher receives a down even after

     *  uptime exceeds this, the dispatcher will stop blocking touches for the launch animation,

     *  which has presumabely not completed due to an error.

     */

    private long mLaunchAnimationTimeout;

    private NotificationStackScrollLayout mStackScrollLayout;

    private PhoneStatusBarViewController mStatusBarViewController;

    private final CentralSurfaces mService;
@@ -266,8 +273,13 @@ public class NotificationShadeWindowViewController { 
                    return logDownDispatch(ev, "touch cancelled", false);

                }

                if (mExpandAnimationRunning) {

                    if (isDown && mClock.uptimeMillis() > mLaunchAnimationTimeout) {

                        mShadeLogger.d("NSWVC: launch animation timed out");

                        setExpandAnimationRunning(false);

                    } else {

                        return logDownDispatch(ev, "expand animation running", false);

                    }

                }



                if (mKeyguardUnlockAnimationController.isPlayingCannedUnlockAnimation()) {

                    // If the user was sliding their finger across the lock screen,
@@ -548,6 +560,9 @@ public class NotificationShadeWindowViewController { 


    public void setExpandAnimationRunning(boolean running) {

        if (mExpandAnimationRunning != running) {

            if (running) {

                mLaunchAnimationTimeout = mClock.uptimeMillis() + 5000;

            }

            mExpandAnimationRunning = running;

            mNotificationShadeWindowController.setLaunchingActivity(mExpandAnimationRunning);

        }

packages/SystemUI/tests/src/com/android/systemui/shade/NotificationShadeWindowViewControllerTest.kt

+31 −1
Original line number Diff line number Diff line
@@ -110,6 +110,7 @@ class NotificationShadeWindowViewControllerTest : SysuiTestCase() { 
    @Mock

    lateinit var primaryBouncerToGoneTransitionViewModel: PrimaryBouncerToGoneTransitionViewModel



    private lateinit var fakeClock: FakeSystemClock

    private lateinit var interactionEventHandlerCaptor: ArgumentCaptor<InteractionEventHandler>

    private lateinit var interactionEventHandler: InteractionEventHandler
@@ -148,6 +149,7 @@ class NotificationShadeWindowViewControllerTest : SysuiTestCase() { 
                    ),

                inputProxy = inputProxy,

            )

        fakeClock = FakeSystemClock()

        underTest =

            NotificationShadeWindowViewController(

                lockscreenShadeTransitionController,
@@ -176,7 +178,7 @@ class NotificationShadeWindowViewControllerTest : SysuiTestCase() { 
                primaryBouncerToGoneTransitionViewModel,

                featureFlags,

                { multiShadeInteractor },

                FakeSystemClock(),

                fakeClock,

                {

                    MultiShadeMotionEventInteractor(

                        applicationContext = context,
@@ -330,6 +332,33 @@ class NotificationShadeWindowViewControllerTest : SysuiTestCase() { 
            assertThat(returnVal).isTrue()

        }



    @Test

    fun handleDispatchTouchEvent_launchAnimationRunningTimesOut() =

        testScope.runTest {

            // GIVEN touch dispatcher in a state that returns true

            underTest.setStatusBarViewController(phoneStatusBarViewController)

            whenever(keyguardUnlockAnimationController.isPlayingCannedUnlockAnimation()).thenReturn(

                true

            )

            assertThat(interactionEventHandler.handleDispatchTouchEvent(DOWN_EVENT)).isTrue()



            // WHEN launch animation is running for 2 seconds

            fakeClock.setUptimeMillis(10000)

            underTest.setExpandAnimationRunning(true)

            fakeClock.advanceTime(2000)



            // THEN touch is ignored

            assertThat(interactionEventHandler.handleDispatchTouchEvent(DOWN_EVENT)).isFalse()



            // WHEN Launch animation is running for 6 seconds

            fakeClock.advanceTime(4000)



            // THEN move is ignored, down is handled, and window is notified

            assertThat(interactionEventHandler.handleDispatchTouchEvent(MOVE_EVENT)).isFalse()

            assertThat(interactionEventHandler.handleDispatchTouchEvent(DOWN_EVENT)).isTrue()

            verify(notificationShadeWindowController).setLaunchingActivity(false)

        }



    @Test

    fun shouldInterceptTouchEvent_statusBarKeyguardViewManagerShouldIntercept() {

        // down event should be intercepted by keyguardViewManager
@@ -350,6 +379,7 @@ class NotificationShadeWindowViewControllerTest : SysuiTestCase() { 


    companion object {

        private val DOWN_EVENT = MotionEvent.obtain(0L, 0L, MotionEvent.ACTION_DOWN, 0f, 0f, 0)

        private val MOVE_EVENT = MotionEvent.obtain(0L, 0L, MotionEvent.ACTION_MOVE, 0f, 0f, 0)

        private const val VIEW_BOTTOM = 100

    }

}