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

Commit fb442a70 authored by Vinit Nayak's avatar Vinit Nayak
Browse files

Set mLastRectRotation to valid rotation 

mLastRectRotation's value is used after
swipe gestures are complete to reset the
layout for overview (after swiping to home
animation is complete), so we need to set
it to a valid value instead of -1.
OverviewActions visibility depends on
it being rotation 0 for it to be visible,
otherwise it won't show up.

Bug: 155844780
Test: Swipe up to home from an app,
Swipe up to overview, observe overview
actions show in portrait.
Swipe up and back down from landscape app,
hit back. Open portrait app, ensure back
gesture still works.

Change-Id: Ia923a54e68af567b4470841783328528fffebe6a
parent 2a4dce14

quickstep/src/com/android/quickstep/OrientationTouchTransformer.java

+4 −2
Original line number Diff line number Diff line
@@ -157,8 +157,9 @@ class OrientationTouchTransformer { 
                mMode != SysUINavigationMode.Mode.TWO_BUTTONS;

        if (mEnableMultipleRegions) {

            mQuickStepStartingRotation = info.rotation;

        } else if (!enableMultipleRegions) {

            mLastRectRotation = mQuickStepStartingRotation = QUICKSTEP_ROTATION_UNINITIALIZED;

        } else {

            mLastRectRotation = 0;

            mQuickStepStartingRotation = QUICKSTEP_ROTATION_UNINITIALIZED;

        }

        resetSwipeRegions(info);

    }
@@ -293,6 +294,7 @@ class OrientationTouchTransformer { 
                        mLastRectTouched = rect;

                        mLastRectRotation = rect.mRotation;

                        if (mEnableMultipleRegions && mCurrentDisplayRotation == mLastRectRotation) {

                            // TODO(b/154580671) might make this block unnecessary

                            // Start a touch session for the default nav region for the display

                            mQuickStepStartingRotation = mLastRectTouched.mRotation;

                            resetSwipeRegions();