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

Commit de3321d1 authored by Sid Soundararajan's avatar Sid Soundararajan
Browse files

Move onStart update of recents tasks to onResume

This is needed to ensure that the update happens between app
launches, since translucent apps do not get to onStop(). Part of fixes
to improve performance for bug.

BUG: 28371792
Change-Id: I47e3a20bba4f006bfc637635d9c9af697a7fc648
parent d6ea95a4
Loading
Loading
Loading
Loading
+16 −18
Original line number Diff line number Diff line
@@ -334,7 +334,23 @@ public class RecentsTvActivity extends Activity implements OnPreDrawListener {
    @Override
    protected void onStart() {
        super.onStart();
        updatePipUI();
    }

    @Override
    public void onEnterAnimationComplete() {
        super.onEnterAnimationComplete();
        if(mLaunchedFromHome) {
            mHomeRecentsEnterExitAnimationHolder.startEnterAnimation(mPipManager.isPipShown());
        }
        mTaskStackViewAdapter.setResetAddedCards(true);
        EventBus.getDefault().send(new EnterRecentsWindowAnimationCompletedEvent());
    }

    @Override
    public void onResume() {
        super.onResume();
        mPipRecentsOverlayManager.onRecentsResumed();
        // Update the recent tasks
        updateRecentsTasks();

@@ -365,24 +381,6 @@ public class RecentsTvActivity extends Activity implements OnPreDrawListener {
        // Notify that recents is now visible
        SystemServicesProxy ssp = Recents.getSystemServices();
        EventBus.getDefault().send(new RecentsVisibilityChangedEvent(this, true));

        updatePipUI();
    }

    @Override
    public void onEnterAnimationComplete() {
        super.onEnterAnimationComplete();
        if(mLaunchedFromHome) {
            mHomeRecentsEnterExitAnimationHolder.startEnterAnimation(mPipManager.isPipShown());
        }
        mTaskStackViewAdapter.setResetAddedCards(true);
        EventBus.getDefault().send(new EnterRecentsWindowAnimationCompletedEvent());
    }

    @Override
    public void onResume() {
        super.onResume();
        mPipRecentsOverlayManager.onRecentsResumed();
        if(mTaskStackHorizontalGridView.getStack().getTaskCount() > 1 && !mLaunchedFromHome) {
            // If there are 2 or more tasks, and we are not launching from home
            // set the selected position to the 2nd task to allow for faster app switching