commit a9dcdf8d0ce8dc56b0f3f2e6a1b619fcde93207f
author Gabriel Peal <gpeal@users.noreply.github.com> Tue Dec 18 13:31:58 2018 -0800
committer GitHub <noreply@github.com> Tue Dec 18 13:31:58 2018 -0800
tree 1b40007bd3a45ffffa16c9c158b4c48ca62d3634
parent 72f25bffb497612f28bbd481a3d7d96b92b56b57

Skip listeners based on keyframes (#1061)

lottie/src/main/java/com/airbnb/lottie/animation/keyframe/BaseKeyframeAnimation.java

diff --git a/lottie/src/main/java/com/airbnb/lottie/animation/keyframe/BaseKeyframeAnimation.java b/lottie/src/main/java/com/airbnb/lottie/animation/keyframe/BaseKeyframeAnimation.java
index 19cf7c2..42c3a7e 100644
--- a/lottie/src/main/java/com/airbnb/lottie/animation/keyframe/BaseKeyframeAnimation.java
+++ b/lottie/src/main/java/com/airbnb/lottie/animation/keyframe/BaseKeyframeAnimation.java
@@ -5,6 +5,7 @@
 import com.airbnb.lottie.value.Keyframe;
 import com.airbnb.lottie.value.LottieValueCallback;
 
+import java.security.Key;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -52,7 +53,7 @@
   public void setProgress(@FloatRange(from = 0f, to = 1f) float progress) {
     // Must use hashCode() since the actual object instance will be returned
     // from getValue() below with the new values.
-    int previousValue = valueCallback == null ? getValue().hashCode() : 0;
+    Keyframe<K> previousKeyframe = getCurrentKeyframe();
     if (progress < getStartDelayProgress()) {
       progress = getStartDelayProgress();
     } else if (progress > getEndProgress()) {
@@ -64,9 +65,9 @@
     }
     this.progress = progress;
     // Just trigger a change but don't compute values if there is a value callback.
-    int newValue = valueCallback == null ? getValue().hashCode() : -1;
+    Keyframe<K> newKeyframe = getCurrentKeyframe();
 
-    if (previousValue != newValue) {
+    if (previousKeyframe != newKeyframe || !newKeyframe.isStatic()) {
       notifyListeners();
     }
   }