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

Commit 55ea9537 authored by Siarhei Vishniakou's avatar Siarhei Vishniakou
Browse files

Sync only if display focus changed 

If the focus didn't change, don't sync. This will allow systems that
have displays with "self-managed focus" not have to sync every time the
user interacts with another display.

Bug: 406311095
Test: presubmit
Flag: EXEMPT small optimization
Change-Id: Ic39254b6afecac813ff3094be23266183b296c4f
parent 0ff724a9

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

+9 −4
Original line number Diff line number Diff line
@@ -3360,16 +3360,20 @@ public class WindowManagerService extends IWindowManager.Stub 


    @Override

    public void moveDisplayToTopIfAllowed(int displayId) {

        moveDisplayToTopInternal(displayId);

        final boolean moved = moveDisplayToTopInternal(displayId);

        if (moved) {

            syncInputTransactions(true /* waitForAnimations */);

        }

    }



    /**

     * Moves the given display to the top. If it cannot be moved to the top this method does

     * nothing (e.g. if the display has the flag FLAG_STEAL_TOP_FOCUS_DISABLED set).

     * @param displayId The display to move to the top.

     *

     * @return whether the move actually occurred.

     */

    void moveDisplayToTopInternal(int displayId) {

    boolean moveDisplayToTopInternal(int displayId) {

        synchronized (mGlobalLock) {

            final DisplayContent displayContent = mRoot.getDisplayContent(displayId);

            if (displayContent != null && mRoot.getTopChild() != displayContent) {
@@ -3379,7 +3383,7 @@ public class WindowManagerService extends IWindowManager.Stub 
                            "Not moving display (displayId=%d) to top. Top focused displayId=%d. "

                                    + "Reason: FLAG_STEAL_TOP_FOCUS_DISABLED",

                            displayId, mRoot.getTopFocusedDisplayContent().getDisplayId());

                    return;

                    return false;

                }



                Transition transition = null;
@@ -3405,6 +3409,7 @@ public class WindowManagerService extends IWindowManager.Stub 
                    transition.setReady(displayContent, true /* ready */);

                }

            }

            return true;

        }

    }