[automerge] DO NOT MERGE Revert "Accept a Window as shown if it's in any transition" 2p: dd2ba83f79

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

Change-Id: Id78824c46966c8bc22849ba89f0978f4d993fe66
This commit is contained in:
Robin Lee
2023-04-14 14:54:47 +00:00
committed by Presubmit Automerger Backend

View File

@@ -2053,19 +2053,16 @@ class WindowState extends WindowContainer<WindowState> implements WindowManagerP
/** /**
* Like isOnScreen(), but we don't return true if the window is part * Like isOnScreen(), but we don't return true if the window is part
* of a transition but has not yet started animating. * of a transition that has not yet been started.
*/ */
boolean isReadyForDisplay() { boolean isReadyForDisplay() {
if (!mHasSurface || mDestroying || !isVisibleByPolicy()) { if (mToken.waitingToShow && getDisplayContent().mAppTransition.isTransitionSet()) {
return false;
}
if (mToken.waitingToShow && getDisplayContent().mAppTransition.isTransitionSet()
&& !isAnimating(TRANSITION | PARENTS, ANIMATION_TYPE_APP_TRANSITION)) {
return false; return false;
} }
final boolean parentAndClientVisible = !isParentWindowHidden() final boolean parentAndClientVisible = !isParentWindowHidden()
&& mViewVisibility == View.VISIBLE && mToken.isVisible(); && mViewVisibility == View.VISIBLE && mToken.isVisible();
return parentAndClientVisible || isAnimating(TRANSITION | PARENTS, ANIMATION_TYPE_ALL); return mHasSurface && isVisibleByPolicy() && !mDestroying
&& (parentAndClientVisible || isAnimating(TRANSITION | PARENTS));
} }
boolean isFullyTransparent() { boolean isFullyTransparent() {