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

Commit 3945f028 authored by Tony Wickham's avatar Tony Wickham
Browse files

Revert line that could wrongly initialize RecentsView scroll 

Test: open a random app that hasn't been opened in a while, touch down
on nav handle and see if RecentsView scrolls partially or fully
offscreen; repeat 20 times to be sure
Bug: 233112195
Fixes: 235369468

Change-Id: Ie8b3dbea7c4826af037edf22360946105ab22134
parent fe83fce4

src/com/android/launcher3/PagedView.java

+3 −1
Original line number Diff line number Diff line
@@ -1187,7 +1187,9 @@ public abstract class PagedView<T extends View & PageIndicator> extends ViewGrou 
    }



    public int getScrollForPage(int index) {

        if (!pageScrollsInitialized() || index >= mPageScrolls.length || index < 0) {

        // TODO(b/233112195): Use !pageScrollsInitialized() instead of mPageScrolls == null, once we

        // root cause where we should be using runOnPageScrollsInitialized().

        if (mPageScrolls == null || index >= mPageScrolls.length || index < 0) {

            return 0;

        } else {

            return mPageScrolls[index];