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

Commit 7d5267cd authored by Evan Rosky's avatar Evan Rosky Committed by Automerger Merge Worker
Browse files

Merge "Don't collect redundant activities in transient launch" into udc-dev... 

Merge "Don't collect redundant activities in transient launch" into udc-dev am: 158198dd am: 8f6deefb

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



Change-Id: If08a9e421744110ed406902aed4e56d6cca243c2
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0ae3f2f4 8f6deefb

services/core/java/com/android/server/wm/Transition.java

+18 −7
Original line number Diff line number Diff line
@@ -287,7 +287,7 @@ class Transition implements BLASTSyncEngine.TransactionReadyListener { 


        if (restoreBelow != null) {

            final Task transientRootTask = activity.getRootTask();

            // Collect all visible activities which can be occluded by the transient activity to

            // Collect all visible tasks which can be occluded by the transient activity to

            // make sure they are in the participants so their visibilities can be updated when

            // finishing transition.

            ((WindowContainer<?>) restoreBelow.getParent()).forAllTasks(t -> {
@@ -297,11 +297,7 @@ class Transition implements BLASTSyncEngine.TransactionReadyListener { 
                        mTransientHideTasks.add(t);

                    }

                    if (t.isLeafTask()) {

                        t.forAllActivities(r -> {

                            if (r.isVisibleRequested()) {

                                collect(r);

                            }

                        });

                        collect(t);

                    }

                }

                return t == restoreBelow;
@@ -904,6 +900,18 @@ class Transition implements BLASTSyncEngine.TransactionReadyListener { 
        mController.mFinishingTransition = this;



        if (mTransientHideTasks != null && !mTransientHideTasks.isEmpty()) {

            // Record all the now-hiding activities so that they are committed after

            // recalculating visibilities. We just use mParticipants because we can and it will

            // ensure proper reporting of `isInFinishTransition`.

            for (int i = 0; i < mTransientHideTasks.size(); ++i) {

                mTransientHideTasks.get(i).forAllActivities(r -> {

                    // Only check leaf-tasks that were collected

                    if (!mParticipants.contains(r.getTask())) return;

                    // Only concern ourselves with anything that can become invisible

                    if (!r.isVisible()) return;

                    mParticipants.add(r);

                });

            }

            // The transient hide tasks could be occluded now, e.g. returning to home. So trigger

            // the update to make the activities in the tasks invisible-requested, then the next

            // step can continue to commit the visibility.
@@ -953,7 +961,10 @@ class Transition implements BLASTSyncEngine.TransactionReadyListener { 
                        enterAutoPip = true;

                    }

                }

                if (mChanges.get(ar).mVisible != visibleAtTransitionEnd) {

                final ChangeInfo changeInfo = mChanges.get(ar);

                // Due to transient-hide, there may be some activities here which weren't in the

                // transition.

                if (changeInfo != null && changeInfo.mVisible != visibleAtTransitionEnd) {

                    // Legacy dispatch relies on this (for now).

                    ar.mEnteringAnimation = visibleAtTransitionEnd;

                } else if (mTransientLaunches != null && mTransientLaunches.containsKey(ar)

services/tests/wmtests/src/com/android/server/wm/TransitionTests.java

+4 −2
Original line number Diff line number Diff line
@@ -1410,9 +1410,9 @@ public class TransitionTests extends WindowTestsBase { 
        final Transition.ChangeInfo task1ChangeInfo = closeTransition.mChanges.get(task1);

        assertNotNull(task1ChangeInfo);

        assertTrue(task1ChangeInfo.hasChanged());

        // Make sure the unrelated activity is NOT collected.

        final Transition.ChangeInfo activity1ChangeInfo = closeTransition.mChanges.get(activity1);

        assertNotNull(activity1ChangeInfo);

        assertTrue(activity1ChangeInfo.hasChanged());

        assertNull(activity1ChangeInfo);

        // No need to wait for the activity in transient hide task.

        assertEquals(WindowContainer.SYNC_STATE_NONE, activity1.mSyncState);
@@ -1442,6 +1442,7 @@ public class TransitionTests extends WindowTestsBase { 
                }

            }

        });

        assertTrue(activity1.isVisible());

        controller.finishTransition(closeTransition);

        assertTrue(wasInFinishingTransition[0]);

        assertNull(controller.mFinishingTransition);
@@ -1450,6 +1451,7 @@ public class TransitionTests extends WindowTestsBase { 
        assertEquals(ActivityTaskManagerService.APP_SWITCH_DISALLOW, mAtm.getBalAppSwitchesState());

        // Because task1 is occluded by task2, finishTransition should make activity1 invisible.

        assertFalse(activity1.isVisibleRequested());

        // Make sure activity1 visibility was committed

        assertFalse(activity1.isVisible());

        assertFalse(activity1.app.hasActivityInVisibleTask());