diff --git a/core/java/android/view/ViewRootImpl.java b/core/java/android/view/ViewRootImpl.java index 38318b604d14f..cb316e27a4da0 100644 --- a/core/java/android/view/ViewRootImpl.java +++ b/core/java/android/view/ViewRootImpl.java @@ -712,6 +712,7 @@ public final class ViewRootImpl implements ViewParent, // These are accessed by multiple threads. final Rect mWinFrame; // frame given by window manager. + private final Rect mLastLayoutFrame; Rect mOverrideInsetsFrame; final Rect mPendingBackDropFrame = new Rect(); @@ -963,6 +964,7 @@ public final class ViewRootImpl implements ViewParent, mHeight = -1; mDirty = new Rect(); mWinFrame = new Rect(); + mLastLayoutFrame = new Rect(); mWindow = new W(this); mLeashToken = new Binder(); mTargetSdkVersion = context.getApplicationInfo().targetSdkVersion; @@ -1329,7 +1331,7 @@ public final class ViewRootImpl implements ViewParent, UNSPECIFIED_LENGTH, UNSPECIFIED_LENGTH, mInsetsController.getRequestedVisibleTypes(), 1f /* compactScale */, mTmpFrames); - setFrame(mTmpFrames.frame); + setFrame(mTmpFrames.frame, true /* withinRelayout */); registerBackCallbackOnWindow(); if (DEBUG_LAYOUT) Log.v(mTag, "Added window " + mWindow); if (res < WindowManagerGlobal.ADD_OKAY) { @@ -1856,7 +1858,7 @@ public final class ViewRootImpl implements ViewParent, onMovedToDisplay(displayId, mLastConfigurationFromResources); } - setFrame(frame); + setFrame(frame, false /* withinRelayout */); mTmpFrames.displayFrame.set(displayFrame); if (mTmpFrames.attachedFrame != null && attachedFrame != null) { mTmpFrames.attachedFrame.set(attachedFrame); @@ -5850,7 +5852,7 @@ public final class ViewRootImpl implements ViewParent, mTmpFrames.frame.right = l + w; mTmpFrames.frame.top = t; mTmpFrames.frame.bottom = t + h; - setFrame(mTmpFrames.frame); + setFrame(mTmpFrames.frame, false /* withinRelayout */); maybeHandleWindowMove(mWinFrame); } break; @@ -8357,7 +8359,7 @@ public final class ViewRootImpl implements ViewParent, // If the position and the size of the frame are both changed, it will trigger a BLAST // sync, and we still need to call relayout to obtain the syncSeqId. Otherwise, we just // need to send attributes via relayoutAsync. - final Rect oldFrame = mWinFrame; + final Rect oldFrame = mLastLayoutFrame; final Rect newFrame = mTmpFrames.frame; final boolean positionChanged = newFrame.top != oldFrame.top || newFrame.left != oldFrame.left; @@ -8487,7 +8489,7 @@ public final class ViewRootImpl implements ViewParent, params.restore(); } - setFrame(mTmpFrames.frame); + setFrame(mTmpFrames.frame, true /* withinRelayout */); return relayoutResult; } @@ -8522,8 +8524,18 @@ public final class ViewRootImpl implements ViewParent, mIsSurfaceOpaque = opaque; } - private void setFrame(Rect frame) { + /** + * Set the mWinFrame of this window. + * @param frame the new frame of this window. + * @param withinRelayout {@code true} if this setting is within the relayout, or is the initial + * setting. That will make sure in the relayout process, we always compare + * the window frame with the last processed window frame. + */ + private void setFrame(Rect frame, boolean withinRelayout) { mWinFrame.set(frame); + if (withinRelayout) { + mLastLayoutFrame.set(frame); + } final WindowConfiguration winConfig = getCompatWindowConfiguration(); mPendingBackDropFrame.set(mPendingDragResizing && !winConfig.useWindowFrameForBackdrop()