From f263e9d30754f868dcd92746c4f2aac86eedd798 Mon Sep 17 00:00:00 2001 From: Johannes Gallmann Date: Tue, 9 Jan 2024 12:04:49 +0100 Subject: [PATCH] Fix TaplTestsQuickstep.testPressback failure (2) Since anim is empty in the predictiveBack case, the animationEnd listener needs to be added to rectFSpringAnim instead Bug: 318675970 Flag: ACONFIG com.android.systemui.predictive_back_system_animations STAGING Test: TaplTestsQuickstep Change-Id: Ibb3fd44c56b9d5370362f994762554eff02edbe6 --- .../android/launcher3/QuickstepTransitionManager.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/quickstep/src/com/android/launcher3/QuickstepTransitionManager.java b/quickstep/src/com/android/launcher3/QuickstepTransitionManager.java index 67d3827dfb..531caa00d0 100644 --- a/quickstep/src/com/android/launcher3/QuickstepTransitionManager.java +++ b/quickstep/src/com/android/launcher3/QuickstepTransitionManager.java @@ -1654,14 +1654,20 @@ public class QuickstepTransitionManager implements OnDeviceProfileChangeListener ? Cuj.CUJ_LAUNCHER_APP_CLOSE_TO_HOME_FALLBACK : Cuj.CUJ_LAUNCHER_APP_CLOSE_TO_HOME); - anim.addListener(new AnimatorListenerAdapter() { + AnimatorListenerAdapter endListener = new AnimatorListenerAdapter() { @Override public void onAnimationEnd(Animator animation) { super.onAnimationEnd(animation); AccessibilityManagerCompat.sendTestProtocolEventToTest( mLauncher, WALLPAPER_OPEN_ANIMATION_FINISHED_MESSAGE); } - }); + }; + + if (fromPredictiveBack) { + rectFSpringAnim.addAnimatorListener(endListener); + } else { + anim.addListener(endListener); + } // Only register the content animation for cancellation when state changes mLauncher.getStateManager().setCurrentAnimation(anim);