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

Commit 1fc54aa0 authored by Vinit Nayak's avatar Vinit Nayak
Browse files

Don't use divider visibility as indicator for split visibility 

* Previously we were using when the divider is visible or not
to notify split visibility listener.
* Now we re-purpose existing setSplitsVisible() method and pass
that value along

Test: Manual, logging
Fixes: 409657406
Flag: EXEMPT bugfix
Change-Id: I1740ddb1ae42d283f546f06a5b3795c034472475
parent f8f213e1

libs/WindowManager/Shell/src/com/android/wm/shell/splitscreen/StageCoordinator.java

+4 −3
Original line number Diff line number Diff line
@@ -2141,12 +2141,13 @@ public class StageCoordinator implements SplitLayout.SplitLayoutHandler, 
        });

    }



    private void sendSplitVisibilityChanged() {

    /** Notify external parties when split is visible or not. NOT related to split activation. */

    private void sendSplitVisibilityChanged(boolean visible) {

        ProtoLog.d(WM_SHELL_SPLIT_SCREEN, "sendSplitVisibilityChanged: dividerVisible=%b",

                mDividerVisible);

        for (int i = mListeners.size() - 1; i >= 0; --i) {

            final SplitScreen.SplitScreenListener l = mListeners.get(i);

            l.onSplitVisibilityChanged(mDividerVisible);

            l.onSplitVisibilityChanged(visible);

        }

        sendOnBoundsChanged();

    }
@@ -2364,7 +2365,6 @@ public class StageCoordinator implements SplitLayout.SplitLayoutHandler, 
        }



        mDividerVisible = visible;

        sendSplitVisibilityChanged();



        if (mIsDividerRemoteAnimating) {

            ProtoLog.d(WM_SHELL_SPLIT_SCREEN,
@@ -4185,6 +4185,7 @@ public class StageCoordinator implements SplitLayout.SplitLayoutHandler, 
        }



        mStatusBarHider.onSplitVisibilityChanged(visible);

        sendSplitVisibilityChanged(visible);

    }



    /**