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

Commit 4ec3d700 authored by Chavi Weingarten's avatar Chavi Weingarten Committed by Android Build Coastguard Worker
Browse files

Set mAlreadyRequestedSync only when in a BLASTSync 

WindowState#useBLASTSync returns true if in a blast sync but also if
there are mDrawHandlers. In that case, we only want to mark
mAlreadyRequestedSync to true if we know it's a blast sync since the
unmark is done when the sync is complete.

Test: Pip expand no delay
Fixes: 236882306
Change-Id: Ifb5785b93888a0fb3e27180005ea1b3940a293b7
(cherry picked from commit 5ef2940b)
Merged-In: Ifb5785b93888a0fb3e27180005ea1b3940a293b7
parent a54f777f

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

+6 −1
Original line number Diff line number Diff line
@@ -122,6 +122,7 @@ import static com.android.server.wm.SurfaceAnimator.ANIMATION_TYPE_ALL; 
import static com.android.server.wm.SurfaceAnimator.ANIMATION_TYPE_APP_TRANSITION;

import static com.android.server.wm.WindowContainer.AnimationFlags.CHILDREN;

import static com.android.server.wm.WindowContainer.AnimationFlags.TRANSITION;

import static com.android.server.wm.WindowContainer.SYNC_STATE_NONE;

import static com.android.server.wm.WindowManagerDebugConfig.DEBUG;

import static com.android.server.wm.WindowManagerDebugConfig.DEBUG_DISPLAY;

import static com.android.server.wm.WindowManagerDebugConfig.DEBUG_INPUT_METHOD;
@@ -2557,7 +2558,11 @@ public class WindowManagerService extends IWindowManager.Stub 


                win.mLastSeqIdSentToRelayout = win.mSyncSeqId;

                outSyncIdBundle.putInt("seqid", win.mSyncSeqId);

                // Only mark mAlreadyRequestedSync if there's an explicit sync request, and not if

                // we're syncing due to mDrawHandlers

                if (win.mSyncState != SYNC_STATE_NONE) {

                    win.mAlreadyRequestedSync = true;

                }

            } else {

                outSyncIdBundle.putInt("seqid", -1);

            }