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

Commit 2f3aa427 authored by Perry Wu's avatar Perry Wu
Browse files

Only update home visibility on exit pip to fullscreen 

In ag/27443168 we update home visibility when expanding pip to
fix a bug with overview. However this introduced a new bug where
taskbar would appear when home was not visible.

This fixes the second issue by only updating home visibility when
we actually expand pip.

Flag: N/A
Bug: 340443311
Test: Manually verify taskbar doesn't appear when dismissing pip,
and that we have not regressed b/340443311 when expanding pip

Change-Id: I341118278faebf446800f89572176a3e52b5bb95
parent 31511247

libs/WindowManager/Shell/src/com/android/wm/shell/pip/PipTransition.java

+3 −3
Original line number Diff line number Diff line
@@ -200,9 +200,6 @@ public class PipTransition extends PipTransitionController { 
            animator.cancel();

        }

        mExitTransition = mTransitions.startTransition(type, out, this);

        if (mPipOrganizer.getOutPipWindowingMode() == WINDOWING_MODE_UNDEFINED) {

            mHomeTransitionObserver.notifyHomeVisibilityChanged(false /* isVisible */);

        }

    }



    @Override
@@ -659,6 +656,9 @@ public class PipTransition extends PipTransitionController { 
            startTransaction.remove(mPipOrganizer.mPipOverlay);

            mPipOrganizer.clearContentOverlay();

        }

        if (mPipOrganizer.getOutPipWindowingMode() == WINDOWING_MODE_UNDEFINED) {

            mHomeTransitionObserver.notifyHomeVisibilityChanged(false /* isVisible */);

        }

        if (pipChange == null) {

            ProtoLog.w(ShellProtoLogGroup.WM_SHELL_PICTURE_IN_PICTURE,

                    "%s: No window of exiting PIP is found. Can't play expand animation", TAG);