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

Commit 9c04429c authored by Tiger Huang's avatar Tiger Huang Committed by Automerger Merge Worker
Browse files

Merge "Don't cache shouldAttachNavBarToAppDuringTransition" into udc-dev am: 72291911 

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



Change-Id: Ic7d5558d58e4e8eb424c65da2372ead62ed9d1a9
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents aa89e13f 72291911

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

+2 −6
Original line number Diff line number Diff line
@@ -146,8 +146,6 @@ public class RecentsAnimationController implements DeathRecipient { 


    @VisibleForTesting

    boolean mIsAddingTaskToTargets;

    @VisibleForTesting

    boolean mShouldAttachNavBarToAppDuringTransition;

    private boolean mNavigationBarAttachedToApp;

    private ActivityRecord mNavBarAttachedApp;
@@ -379,8 +377,6 @@ public class RecentsAnimationController implements DeathRecipient { 
        mDisplayId = displayId;

        mStatusBar = LocalServices.getService(StatusBarManagerInternal.class);

        mDisplayContent = service.mRoot.getDisplayContent(displayId);

        mShouldAttachNavBarToAppDuringTransition =

                mDisplayContent.getDisplayPolicy().shouldAttachNavBarToAppDuringTransition();

    }



    /**
@@ -577,7 +573,7 @@ public class RecentsAnimationController implements DeathRecipient { 
    }



    private void attachNavigationBarToApp() {

        if (!mShouldAttachNavBarToAppDuringTransition

        if (!mDisplayContent.getDisplayPolicy().shouldAttachNavBarToAppDuringTransition()

                // Skip the case where the nav bar is controlled by fade rotation.

                || mDisplayContent.getAsyncRotationController() != null) {

            return;
@@ -652,7 +648,7 @@ public class RecentsAnimationController implements DeathRecipient { 
    }



    void animateNavigationBarForAppLaunch(long duration) {

        if (!mShouldAttachNavBarToAppDuringTransition

        if (!mDisplayContent.getDisplayPolicy().shouldAttachNavBarToAppDuringTransition()

                // Skip the case where the nav bar is controlled by fade rotation.

                || mDisplayContent.getAsyncRotationController() != null

                || mNavigationBarAttachedToApp

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

+1 −2
Original line number Diff line number Diff line
@@ -104,7 +104,6 @@ public class RecentsAnimationControllerTest extends WindowTestsBase { 
        when(mMockRunner.asBinder()).thenReturn(new Binder());

        mController = spy(new RecentsAnimationController(mWm, mMockRunner, mAnimationCallbacks,

                DEFAULT_DISPLAY));

        mController.mShouldAttachNavBarToAppDuringTransition = false;

        mRootHomeTask = mDefaultDisplay.getDefaultTaskDisplayArea().getRootHomeTask();

        assertNotNull(mRootHomeTask);

    }
@@ -814,13 +813,13 @@ public class RecentsAnimationControllerTest extends WindowTestsBase { 
    }



    private void setupForShouldAttachNavBarDuringTransition() {

        mController.mShouldAttachNavBarToAppDuringTransition = true;

        final WindowState navBar = spy(createWindow(null, TYPE_NAVIGATION_BAR, "NavigationBar"));

        mDefaultDisplay.getDisplayPolicy().addWindowLw(navBar, navBar.mAttrs);

        mWm.setRecentsAnimationController(mController);

        doReturn(navBar).when(mController).getNavigationBarWindow();

        final DisplayPolicy displayPolicy = spy(mDefaultDisplay.getDisplayPolicy());

        doReturn(displayPolicy).when(mDefaultDisplay).getDisplayPolicy();

        doReturn(true).when(displayPolicy).shouldAttachNavBarToAppDuringTransition();

    }



    private static void initializeRecentsAnimationController(RecentsAnimationController controller,

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

+4 −1
Original line number Diff line number Diff line
@@ -500,7 +500,6 @@ public class ZOrderingTests extends WindowTestsBase { 
        RecentsAnimationController controller = new RecentsAnimationController(

                mWm, mockRunner, null, displayId);

        spyOn(controller);

        controller.mShouldAttachNavBarToAppDuringTransition = true;

        doReturn(mNavBarWindow).when(controller).getNavigationBarWindow();

        mWm.setRecentsAnimationController(controller);
@@ -508,6 +507,10 @@ public class ZOrderingTests extends WindowTestsBase { 
        spyOn(mDisplayContent.mInputMethodWindow);

        doReturn(true).when(mDisplayContent.mInputMethodWindow).isVisible();



        DisplayPolicy policy = mDisplayContent.getDisplayPolicy();

        spyOn(policy);

        doReturn(true).when(policy).shouldAttachNavBarToAppDuringTransition();



        // create home activity

        Task rootHomeTask = mDisplayContent.getDefaultTaskDisplayArea().getRootHomeTask();

        final ActivityRecord homeActivity = new ActivityBuilder(mWm.mAtmService)