summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Evan Rosky <erosky@google.com> 2021-08-30 12:12:11 -0700
committer Evan Rosky <erosky@google.com> 2021-08-30 12:16:24 -0700
commit2c0789b8f82ae2ac2b1713a62098e87f4adb6d3c (patch)
treec9abc2afc54bdfe67890acd5cab62d410545afad
parent81b968f843b6316b93b434ef0f64ac7528e46a1d (diff)
Don't explicitly wait for starting-window during tests with shell-transitions
Starting-window is part of sync-set when using shell-transitions, so don't wait for it separately in waitForAnimationsToComplete because it will just time-out. Bug: 183993924 Test: atest WindowUntrustedTouchTests Change-Id: I4788148dab3e6b957a7d37dee01587b904d46e48
-rw-r--r--services/core/java/com/android/server/wm/WindowManagerService.java5
1 files changed, 2 insertions, 3 deletions
diff --git a/services/core/java/com/android/server/wm/WindowManagerService.java b/services/core/java/com/android/server/wm/WindowManagerService.java
index 43b555c914bc..4f3c6d5b6b5f 100644
--- a/services/core/java/com/android/server/wm/WindowManagerService.java
+++ b/services/core/java/com/android/server/wm/WindowManagerService.java
@@ -8111,9 +8111,8 @@ public class WindowManagerService extends IWindowManager.Stub
boolean animateStarting = false;
while (timeoutRemaining > 0) {
// Waiting until all starting windows has finished animating.
- animateStarting = mRoot.forAllActivities(a -> {
- return a.hasStartingWindow();
- });
+ animateStarting = !mAtmService.getTransitionController().isShellTransitionsEnabled()
+ && mRoot.forAllActivities(ActivityRecord::hasStartingWindow);
boolean isAnimating = mAnimator.isAnimationScheduled()
|| mRoot.isAnimating(TRANSITION | CHILDREN, ANIMATION_TYPE_ALL)
|| animateStarting;