From cf1913b4d45a5568031c42a803c69292e8803d84 Mon Sep 17 00:00:00 2001 From: Evan Rosky Date: Mon, 17 Apr 2023 19:11:02 -0700 Subject: [PATCH] Also put show in the finish transaction when setting up animation It's possible for a series of merges to happen on the same task. If only the hide's are represented in the finishT and the startTs were merged, then its possible to leave surfaces hidden when they shouldn't be. Bug: 278590171 Test: make rapid successive quickswitches and swipe-ups. Change-Id: Ifacf5811cb64451013df87f7305f761c5eb78a47 --- .../Shell/src/com/android/wm/shell/transition/Transitions.java | 1 + 1 file changed, 1 insertion(+) diff --git a/libs/WindowManager/Shell/src/com/android/wm/shell/transition/Transitions.java b/libs/WindowManager/Shell/src/com/android/wm/shell/transition/Transitions.java index bdb7d44bad32b..08b0bf74f4131 100644 --- a/libs/WindowManager/Shell/src/com/android/wm/shell/transition/Transitions.java +++ b/libs/WindowManager/Shell/src/com/android/wm/shell/transition/Transitions.java @@ -429,6 +429,7 @@ public class Transitions implements RemoteCallable { && (change.getFlags() & FLAG_STARTING_WINDOW_TRANSFER_RECIPIENT) == 0) { t.setAlpha(leash, 0.f); } + finishT.show(leash); } else if (mode == TRANSIT_CLOSE || mode == TRANSIT_TO_BACK) { finishT.hide(leash); }