cc : Track transform animation changes on transform tree

cc/animation/element_animations.h

Index: cc/animation/element_animations.h
diff --git a/cc/animation/element_animations.h b/cc/animation/element_animations.h
index a3317a09a77ad1c3b233aa71e688cae7f21c000e..1b20c75fb2eb7fde2c22810c04d35263636613f2 100644
--- a/cc/animation/element_animations.h
+++ b/cc/animation/element_animations.h
@@ -224,19 +224,13 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
 
   void NotifyClientAnimationWaitingForDeletion();
 
-  void NotifyClientTransformIsPotentiallyAnimatingChanged(
-      bool notify_active_elements,
-      bool notify_pending_elements);
+  void NotifyClientAnimationChanged(
+      TargetProperty::Type property,
+      ElementListType list_type,
+      bool notify_elements_about_potential_animation,
+      bool notify_elements_about_running_animation);
 
-  void UpdatePotentiallyAnimatingTransform();
-
-  void NotifyClientOpacityAnimationChanged(
-      bool notify_active_elements_about_potential_animation,
-      bool notify_active_elements_about_running_animation,
-      bool notify_pending_elements_about_potential_aniamtion,
-      bool notify_pending_elements_about_running_animation);
-
-  void UpdateAnimatingOpacity();
+  void UpdateAnimationState(TargetProperty::Type property);

[ajuma, 2016/05/18 17:09:12]

This name is confusingly close to the existing UpdateState method. Maybe call it
UpdateClientAnimationState or UpdateAnimatingProperties?

[jaydasika, 2016/05/18 21:42:37]

Done.

 
   void OnFilterAnimated(ElementListType list_type,
                         const FilterOperations& filters);
@@ -246,11 +240,10 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
   void OnScrollOffsetAnimated(ElementListType list_type,
                               const gfx::ScrollOffset& scroll_offset);
   void OnAnimationWaitingForDeletion();
-  void OnTransformIsPotentiallyAnimatingChanged(ElementListType list_type,
-                                                bool is_animating);
-  void OnOpacityIsAnimatingChanged(ElementListType list_type,
-                                   AnimationChangeType change_type,
-                                   bool is_animating);
+  void IsAnimatingChanged(ElementListType list_type,
+                          TargetProperty::Type property,
+                          AnimationChangeType change_type,
+                          bool is_animating);
   gfx::ScrollOffset ScrollOffsetForAnimation() const;
 
   void NotifyPlayersAnimationStarted(base::TimeTicks monotonic_time,
@@ -287,6 +280,8 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
 
   bool scroll_offset_animation_was_interrupted_;
 
+  bool currently_running_transform_animation_for_active_elements_;
+  bool currently_running_transform_animation_for_pending_elements_;
   bool potentially_animating_transform_for_active_elements_;
   bool potentially_animating_transform_for_pending_elements_;