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

Commit 17cdb752 authored by Christian Göllner's avatar Christian Göllner Committed by Automerger Merge Worker
Browse files

Merge "Fix QS and QS Footer sometimes having low alpha after expansion" into... 

Merge "Fix QS and QS Footer sometimes having low alpha after expansion" into tm-qpr-dev am: 23cb555f am: 0e203be0

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



Change-Id: I0aed37cccbfb95a16616b66f5d8b06dec59d4f4a
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 51537e59 0e203be0

packages/SystemUI/src/com/android/systemui/qs/QSFragment.java

+2 −1
Original line number Diff line number Diff line
@@ -635,7 +635,8 @@ public class QSFragment extends LifecycleFragment implements QS, CommandQueue.Ca 
                && mLastKeyguardAndExpanded == onKeyguardAndExpanded

                && mLastViewHeight == currentHeight

                && mLastHeaderTranslation == headerTranslation

                && mSquishinessFraction == squishinessFraction) {

                && mSquishinessFraction == squishinessFraction

                && mLastPanelFraction == panelExpansionFraction) {

            return;

        }

        mLastHeaderTranslation = headerTranslation;

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

+2 −2
Original line number Diff line number Diff line
@@ -3779,10 +3779,10 @@ public final class NotificationPanelViewController implements Dumpable { 
                    mHeightAnimator.end();

                }

            }

            mQsController.setShadeExpandedHeight(mExpandedHeight);

            mExpansionDragDownAmountPx = h;

            mExpandedFraction = Math.min(1f,

                    maxPanelHeight == 0 ? 0 : mExpandedHeight / maxPanelHeight);

            mQsController.setShadeExpansion(mExpandedHeight, mExpandedFraction);

            mExpansionDragDownAmountPx = h;

            mAmbientState.setExpansionFraction(mExpandedFraction);

            onHeightUpdated(mExpandedHeight);

            updatePanelExpansionAndVisibility();

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

+3 −8
Original line number Diff line number Diff line
@@ -351,7 +351,6 @@ public class QuickSettingsController { 
        mFeatureFlags = featureFlags;

        mInteractionJankMonitor = interactionJankMonitor;



        mShadeExpansionStateManager.addExpansionListener(this::onPanelExpansionChanged);

        mLockscreenShadeTransitionController.addCallback(new LockscreenShadeTransitionCallback());

    }
@@ -878,8 +877,9 @@ public class QuickSettingsController { 
        mCollapsedOnDown = collapsedOnDown;

    }



    void setShadeExpandedHeight(float shadeExpandedHeight) {

        mShadeExpandedHeight = shadeExpandedHeight;

    void setShadeExpansion(float expandedHeight, float expandedFraction) {

        mShadeExpandedHeight = expandedHeight;

        mShadeExpandedFraction = expandedFraction;

    }



    @VisibleForTesting
@@ -1749,11 +1749,6 @@ public class QuickSettingsController { 
        return false;

    }



    @VisibleForTesting

    void onPanelExpansionChanged(ShadeExpansionChangeEvent event) {

        mShadeExpandedFraction = event.getFraction();

    }



    /**

     * Animate QS closing by flinging it.

     * If QS is expanded, it will collapse into QQS and stop.

packages/SystemUI/tests/src/com/android/systemui/shade/QuickSettingsControllerTest.java

+7 −14
Original line number Diff line number Diff line
@@ -275,9 +275,7 @@ public class QuickSettingsControllerTest extends SysuiTestCase { 


    @Test

    public void testPanelStaysOpenWhenClosingQs() {

        mShadeExpansionStateManager.onPanelExpansionChanged(/* fraction= */ 1,

                /* expanded= */ true, /* tracking= */ false, /* dragDownPxAmount= */ 0);

        mQsController.setShadeExpandedHeight(1);

        mQsController.setShadeExpansion(/* shadeExpandedHeight= */ 1, /* expandedFraction=*/ 1);



        float shadeExpandedHeight = mQsController.getShadeExpandedHeight();

        mQsController.animateCloseQs(false);
@@ -289,7 +287,7 @@ public class QuickSettingsControllerTest extends SysuiTestCase { 
    public void interceptTouchEvent_withinQs_shadeExpanded_startsQsTracking() {

        mQsController.setQs(mQs);



        mQsController.setShadeExpandedHeight(1f);

        mQsController.setShadeExpansion(/* shadeExpandedHeight= */ 1, /* expandedFraction=*/ 1);

        mQsController.onIntercept(

                createMotionEvent(0, 0, ACTION_DOWN));

        mQsController.onIntercept(
@@ -303,7 +301,7 @@ public class QuickSettingsControllerTest extends SysuiTestCase { 
        enableSplitShade(true);

        mQsController.setQs(mQs);



        mQsController.setShadeExpandedHeight(1f);

        mQsController.setShadeExpansion(/* shadeExpandedHeight= */ 1, /* expandedFraction=*/ 1);

        mQsController.onIntercept(

                createMotionEvent(0, 0, ACTION_DOWN));

        mQsController.onIntercept(
@@ -342,13 +340,8 @@ public class QuickSettingsControllerTest extends SysuiTestCase { 
    public void handleTouch_downActionInQsArea() {

        mQsController.setQs(mQs);

        mQsController.setBarState(SHADE);

        mQsController.onPanelExpansionChanged(

                new ShadeExpansionChangeEvent(

                        0.5f,

                        true,

                        true,

                        0

                ));

        mQsController.setShadeExpansion(/* shadeExpandedHeight= */ 1, /* expandedFraction=*/ 0.5f);



        MotionEvent event =

                createMotionEvent(QS_FRAME_WIDTH / 2, QS_FRAME_BOTTOM / 2, ACTION_DOWN);

        mQsController.handleTouch(event, false, false);
@@ -385,7 +378,7 @@ public class QuickSettingsControllerTest extends SysuiTestCase { 
    @Test

    public void handleTouch_isConflictingExpansionGestureSet() {

        assertThat(mQsController.isConflictingExpansionGesture()).isFalse();

        mShadeExpansionStateManager.onPanelExpansionChanged(1f, true, false, 0f);

        mQsController.setShadeExpansion(/* shadeExpandedHeight= */ 1, /* expandedFraction=*/ 1);

        mQsController.handleTouch(MotionEvent.obtain(0L /* downTime */,

                0L /* eventTime */, ACTION_DOWN, 0f /* x */, 0f /* y */,

                0 /* metaState */), false, false);
@@ -394,7 +387,7 @@ public class QuickSettingsControllerTest extends SysuiTestCase { 


    @Test

    public void handleTouch_isConflictingExpansionGestureSet_cancel() {

        mShadeExpansionStateManager.onPanelExpansionChanged(1f, true, false, 0f);

        mQsController.setShadeExpansion(/* shadeExpandedHeight= */ 1, /* expandedFraction=*/ 1);

        mQsController.handleTouch(createMotionEvent(0, 0, ACTION_DOWN), false, false);

        assertThat(mQsController.isConflictingExpansionGesture()).isTrue();

        mQsController.handleTouch(createMotionEvent(0, 0, ACTION_UP), true, true);