commit 52bbcc3976576127c757001485d20416c6f8b36e
author Tony Wickham <twickham@google.com> Thu Mar 25 11:22:21 2021 -0700
committer Tony Wickham <twickham@google.com> Mon Mar 29 11:59:14 2021 -0700
tree 4a189c4bdf70c7420bea41b12382cd4228389784
parent 6cd95cd2d823978edd98c7e605bf2d3f12c1aa37

Invert StateAnimationConfig.PLAY_ANIMATION as SKIP_ALL_ANIMATIONS

This allows the default flag to be 0 instead of PLAY_ANIMATION, and is
aligned with the existing SKIP_OVERVIEW and SKIP_DEPTH_CONTROLLER
flags.

Test: Navigate to various states, works as before
Bug: 175137718
Change-Id: I2af1792e7fbd5bca82afb225290fd6b545368dcf

src/com/android/launcher3/states/StateAnimationConfig.java

diff --git a/src/com/android/launcher3/states/StateAnimationConfig.java b/src/com/android/launcher3/states/StateAnimationConfig.java
index 2045733..8e7dcc0 100644
--- a/src/com/android/launcher3/states/StateAnimationConfig.java
+++ b/src/com/android/launcher3/states/StateAnimationConfig.java
@@ -28,20 +28,19 @@
 public class StateAnimationConfig {
 
     @IntDef(flag = true, value = {
-            PLAY_ANIMATION,
+            SKIP_ALL_ANIMATIONS,
             SKIP_OVERVIEW,
             SKIP_DEPTH_CONTROLLER,
     })
     @Retention(RetentionPolicy.SOURCE)
     public @interface AnimationFlags {}
-    // TODO: make this the default; invert this to be SKIP_ALL_ANIMATIONS
-    public static final int PLAY_ANIMATION = 1 << 0;
+    public static final int SKIP_ALL_ANIMATIONS = 1 << 0;
     public static final int SKIP_OVERVIEW = 1 << 1;
     public static final int SKIP_DEPTH_CONTROLLER = 1 << 2;
 
     public long duration;
     public boolean userControlled;
-    public @AnimationFlags int animFlags = PLAY_ANIMATION;
+    public @AnimationFlags int animFlags = 0;
 
 
     // Various types of animation state transition