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

Commit 849235c8 authored by Lyn Han's avatar Lyn Han Committed by Automerger Merge Worker
Browse files

Merge "Fix white line before shelf on lockscreen" into tm-dev am: 565c12dd am: ed0976bf 

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



Change-Id: Iaa549961520a906a53c6dd799a7715166cd70867
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 60cf6be1 ed0976bf

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackScrollLayout.java

+5 −5
Original line number Diff line number Diff line
@@ -191,7 +191,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
    private final boolean mDebugRemoveAnimation;



    private int mContentHeight;

    private int mIntrinsicContentHeight;

    private float mIntrinsicContentHeight;

    private int mCollapsedSize;

    private int mPaddingBetweenElements;

    private int mMaxTopPadding;
@@ -802,7 +802,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
        drawDebugInfo(canvas, y, Color.MAGENTA,

                /* label= */ "mAmbientState.getStackY() + mContentHeight = " + y);



        y = (int) mAmbientState.getStackY() + mIntrinsicContentHeight;

        y = (int) (mAmbientState.getStackY() + mIntrinsicContentHeight);

        drawDebugInfo(canvas, y, Color.YELLOW,

                /* label= */ "mAmbientState.getStackY() + mIntrinsicContentHeight = " + y);
@@ -1473,7 +1473,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
     */

    @ShadeViewRefactor(RefactorComponent.COORDINATOR)

    public int getIntrinsicContentHeight() {

        return mIntrinsicContentHeight;

        return (int) mIntrinsicContentHeight;

    }



    @ShadeViewRefactor(RefactorComponent.STATE_RESOLVER)
@@ -2287,7 +2287,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
    private void updateContentHeight() {

        final float scrimTopPadding = mAmbientState.isOnKeyguard() ? 0 : mMinimumPaddings;

        final int shelfIntrinsicHeight = mShelf != null ? mShelf.getIntrinsicHeight() : 0;

        final int height =

        final float height =

                (int) scrimTopPadding + (int) mNotificationStackSizeCalculator.computeHeight(

                        /* notificationStackScrollLayout= */ this, mMaxDisplayedNotifications,

                        shelfIntrinsicHeight);
@@ -2295,7 +2295,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 


        // The topPadding can be bigger than the regular padding when qs is expanded, in that

        // state the maxPanelHeight and the contentHeight should be bigger

        mContentHeight = height + Math.max(mIntrinsicPadding, mTopPadding) + mBottomPadding;

        mContentHeight = (int) (height + Math.max(mIntrinsicPadding, mTopPadding) + mBottomPadding);

        updateScrollability();

        clampScrollPosition();

        updateStackPosition();

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/StackScrollAlgorithm.java

+27 −22
Original line number Diff line number Diff line
@@ -53,9 +53,9 @@ public class StackScrollAlgorithm { 
    private static final Boolean DEBUG = false;



    private final ViewGroup mHostView;

    private int mPaddingBetweenElements;

    private int mGapHeight;

    private int mGapHeightOnLockscreen;

    private float mPaddingBetweenElements;

    private float mGapHeight;

    private float mGapHeightOnLockscreen;

    private int mCollapsedSize;



    private StackScrollAlgorithmState mTempAlgorithmState = new StackScrollAlgorithmState();
@@ -127,13 +127,13 @@ public class StackScrollAlgorithm { 
        return getExpansionFractionWithoutShelf(mTempAlgorithmState, ambientState);

    }



    private void log(String s) {

    public static void log(String s) {

        if (DEBUG) {

            android.util.Log.i(TAG, s);

        }

    }



    public void logView(View view, String s) {

    public static void logView(View view, String s) {

        String viewString = "";

        if (view instanceof ExpandableNotificationRow) {

            ExpandableNotificationRow row = ((ExpandableNotificationRow) view);
@@ -535,30 +535,23 @@ public class StackScrollAlgorithm { 
                    //  more notifications than we should during this special transitional states.

                    boolean bypassPulseNotExpanding = ambientState.isBypassEnabled()

                            && ambientState.isOnKeyguard() && !ambientState.isPulseExpanding();

                    final int stackBottom =

                            !ambientState.isShadeExpanded() || ambientState.isDozing()

                    final float stackBottom = !ambientState.isShadeExpanded()

                            || ambientState.getDozeAmount() == 1f

                            || bypassPulseNotExpanding

                                    ? ambientState.getInnerHeight()

                                    : (int) ambientState.getStackHeight();

                    final int shelfStart = stackBottom

                                    : ambientState.getStackHeight();

                    final float shelfStart = stackBottom

                            - ambientState.getShelf().getIntrinsicHeight()

                            - mPaddingBetweenElements;

                    viewState.yTranslation = Math.min(viewState.yTranslation, shelfStart);

                    if (viewState.yTranslation >= shelfStart) {

                        viewState.hidden = !view.isExpandAnimationRunning()

                                && !view.hasExpandingChild();

                        viewState.inShelf = true;

                        // Notifications in the shelf cannot be visible HUNs.

                        viewState.headsUpIsVisible = false;

                    }

                    updateViewWithShelf(view, viewState, shelfStart);

                }

            }

            // Clip height of view right before shelf.

            viewState.height = (int) (getMaxAllowedChildHeight(view) * expansionFraction);

        }



        algorithmState.mCurrentYPosition += viewState.height

                + expansionFraction * mPaddingBetweenElements;

        algorithmState.mCurrentYPosition +=

                expansionFraction * (getMaxAllowedChildHeight(view) + mPaddingBetweenElements);

        algorithmState.mCurrentExpandedYPosition += view.getIntrinsicHeight()

                + mPaddingBetweenElements;
@@ -566,6 +559,18 @@ public class StackScrollAlgorithm { 
        viewState.yTranslation += ambientState.getStackY();

    }



    @VisibleForTesting

    void updateViewWithShelf(ExpandableView view, ExpandableViewState viewState, float shelfStart) {

        viewState.yTranslation = Math.min(viewState.yTranslation, shelfStart);

        if (viewState.yTranslation >= shelfStart) {

            viewState.hidden = !view.isExpandAnimationRunning()

                    && !view.hasExpandingChild();

            viewState.inShelf = true;

            // Notifications in the shelf cannot be visible HUNs.

            viewState.headsUpIsVisible = false;

        }

    }



    /**

     * Get the gap height needed for before a view

     *
@@ -849,13 +854,13 @@ public class StackScrollAlgorithm { 
         * Y position of the current view during updating children

         * with expansion factor applied.

         */

        private int mCurrentYPosition;

        private float mCurrentYPosition;



        /**

         * Y position of the current view during updating children

         * without applying the expansion factor.

         */

        private int mCurrentExpandedYPosition;

        private float mCurrentExpandedYPosition;

    }



    /**

packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/stack/StackScrollAlgorithmTest.kt

+51 −0
Original line number Diff line number Diff line
@@ -7,10 +7,13 @@ import com.android.systemui.R 
import com.android.systemui.SysuiTestCase

import com.android.systemui.statusbar.EmptyShadeView

import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow

import com.android.systemui.statusbar.notification.row.ExpandableView

import com.android.systemui.statusbar.notification.stack.StackScrollAlgorithm.BypassController

import com.android.systemui.statusbar.notification.stack.StackScrollAlgorithm.SectionProvider

import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager

import com.google.common.truth.Truth.assertThat

import junit.framework.Assert.assertFalse

import junit.framework.Assert.assertTrue

import org.junit.Before

import org.junit.Test

import org.mockito.Mockito.mock
@@ -110,4 +113,52 @@ class StackScrollAlgorithmTest : SysuiTestCase() { 
                /* fractionToShade= */ 0f, /* onKeyguard= */ false)

        assertThat(gap).isEqualTo(bigGap)

    }



    @Test

    fun updateViewWithShelf_viewAboveShelf_viewShown() {

        val viewStart = 0f

        val shelfStart = 1f



        val expandableView = mock(ExpandableView::class.java)

        whenever(expandableView.isExpandAnimationRunning).thenReturn(false)

        whenever(expandableView.hasExpandingChild()).thenReturn(false)



        val expandableViewState = ExpandableViewState()

        expandableViewState.yTranslation = viewStart



        stackScrollAlgorithm.updateViewWithShelf(expandableView, expandableViewState, shelfStart);

        assertFalse(expandableViewState.hidden)

    }



    @Test

    fun updateViewWithShelf_viewBelowShelf_viewHidden() {

        val shelfStart = 0f

        val viewStart = 1f



        val expandableView = mock(ExpandableView::class.java)

        whenever(expandableView.isExpandAnimationRunning).thenReturn(false)

        whenever(expandableView.hasExpandingChild()).thenReturn(false)



        val expandableViewState = ExpandableViewState()

        expandableViewState.yTranslation = viewStart



        stackScrollAlgorithm.updateViewWithShelf(expandableView, expandableViewState, shelfStart);

        assertTrue(expandableViewState.hidden)

    }



    @Test

    fun updateViewWithShelf_viewBelowShelfButIsExpanding_viewShown() {

        val shelfStart = 0f

        val viewStart = 1f



        val expandableView = mock(ExpandableView::class.java)

        whenever(expandableView.isExpandAnimationRunning).thenReturn(true)

        whenever(expandableView.hasExpandingChild()).thenReturn(true)



        val expandableViewState = ExpandableViewState()

        expandableViewState.yTranslation = viewStart



        stackScrollAlgorithm.updateViewWithShelf(expandableView, expandableViewState, shelfStart);

        assertFalse(expandableViewState.hidden)

    }

}