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

Commit cd70dc3d authored by Tony Huang's avatar Tony Huang Committed by Automerger Merge Worker
Browse files

Merge "Fix split dismissed after click pip to split button" into sc-v2-dev am:...

Merge "Fix split dismissed after click pip to split button" into sc-v2-dev am: 6a2ce827 am: 5611fc6c

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

Change-Id: I231cfa55010874edb1631bbed8dcb0519e8679a7
parents 4f590a93 5611fc6c
Loading
Loading
Loading
Loading
+6 −0
Original line number Diff line number Diff line
@@ -87,6 +87,12 @@ public interface SplitScreen {
     */
    void onKeyguardVisibilityChanged(boolean showing);

    /** Called when device waking up finished. */
    void onFinishedWakingUp();

    /** Called when device going to sleep finished. */
    void onFinishedGoingToSleep();

    /** Get a string representation of a stage type */
    static String stageTypeToString(@StageType int stage) {
        switch (stage) {
+22 −0
Original line number Diff line number Diff line
@@ -232,6 +232,14 @@ public class SplitScreenController implements DragAndDropPolicy.Starter,
        mStageCoordinator.onKeyguardVisibilityChanged(showing);
    }

    public void onFinishedWakingUp() {
        mStageCoordinator.onFinishedWakingUp();
    }

    public void onFinishedGoingToSleep() {
        mStageCoordinator.onFinishedGoingToSleep();
    }

    public void exitSplitScreenOnHide(boolean exitSplitScreenOnHide) {
        mStageCoordinator.exitSplitScreenOnHide(exitSplitScreenOnHide);
    }
@@ -491,6 +499,20 @@ public class SplitScreenController implements DragAndDropPolicy.Starter,
                SplitScreenController.this.onKeyguardVisibilityChanged(showing);
            });
        }

        @Override
        public void onFinishedWakingUp() {
            mMainExecutor.execute(() -> {
                SplitScreenController.this.onFinishedWakingUp();
            });
        }

        @Override
        public void onFinishedGoingToSleep() {
            mMainExecutor.execute(() -> {
                SplitScreenController.this.onFinishedGoingToSleep();
            });
        }
    }

    /**
+26 −7
Original line number Diff line number Diff line
@@ -155,6 +155,7 @@ class StageCoordinator implements SplitLayout.SplitLayoutHandler,
    private boolean mShouldUpdateRecents;
    private boolean mExitSplitScreenOnHide;
    private boolean mKeyguardOccluded;
    private boolean mDeviceSleep;

    @SplitScreen.StageType
    private int mDismissTop = NO_DISMISS;
@@ -547,6 +548,17 @@ class StageCoordinator implements SplitLayout.SplitLayoutHandler,
        }
    }

    void onFinishedWakingUp() {
        if (mMainStage.isActive()) {
            exitSplitScreenIfKeyguardOccluded();
        }
        mDeviceSleep = false;
    }

    void onFinishedGoingToSleep() {
        mDeviceSleep = true;
    }

    void exitSplitScreenOnHide(boolean exitSplitScreenOnHide) {
        mExitSplitScreenOnHide = exitSplitScreenOnHide;
    }
@@ -575,6 +587,19 @@ class StageCoordinator implements SplitLayout.SplitLayoutHandler,
        applyExitSplitScreen(childrenToTop, wct, exitReason);
    }

    private void exitSplitScreenIfKeyguardOccluded() {
        final boolean mainStageVisible = mMainStageListener.mVisible;
        final boolean oneStageVisible = mainStageVisible ^ mSideStageListener.mVisible;
        if (mDeviceSleep && mKeyguardOccluded && oneStageVisible) {
            // Only the stages include show-when-locked activity is visible while keyguard occluded.
            // Dismiss split because there's show-when-locked activity showing on top of keyguard.
            // Also make sure the task contains show-when-locked activity remains on top after split
            // dismissed.
            final StageTaskListener toTop = mainStageVisible ? mMainStage : mSideStage;
            exitSplitScreen(toTop, EXIT_REASON_SCREEN_LOCKED_SHOW_ON_TOP);
        }
    }

    private void applyExitSplitScreen(StageTaskListener childrenToTop,
            WindowContainerTransaction wct, @ExitReason int exitReason) {
        mRecentTasks.ifPresent(recentTasks -> {
@@ -790,14 +815,8 @@ class StageCoordinator implements SplitLayout.SplitLayoutHandler,
            // like the cases keyguard showing or screen off.
                exitSplitScreen(null /* childrenToTop */, EXIT_REASON_RETURN_HOME);
            }
        } else if (mKeyguardOccluded) {
            // At least one of the stages is visible while keyguard occluded. Dismiss split because
            // there's show-when-locked activity showing on top of keyguard. Also make sure the
            // task contains show-when-locked activity remains on top after split dismissed.
            final StageTaskListener toTop =
                    mainStageVisible ? mMainStage : (sideStageVisible ? mSideStage : null);
            exitSplitScreen(toTop, EXIT_REASON_SCREEN_LOCKED_SHOW_ON_TOP);
        }
        exitSplitScreenIfKeyguardOccluded();

        mSyncQueue.runInSync(t -> {
            // Same above, we only set root tasks and divider leash visibility when both stage
+12 −0
Original line number Diff line number Diff line
@@ -260,6 +260,18 @@ public final class WMShell extends CoreStartable
            }
        };
        mKeyguardUpdateMonitor.registerCallback(mSplitScreenKeyguardCallback);

        mWakefulnessLifecycle.addObserver(new WakefulnessLifecycle.Observer() {
            @Override
            public void onFinishedWakingUp() {
                splitScreen.onFinishedWakingUp();
            }

            @Override
            public void onFinishedGoingToSleep() {
                splitScreen.onFinishedGoingToSleep();
            }
        });
    }

    @VisibleForTesting