cc : Track opacity animation changes on effect tree

cc/layers/layer_impl.cc

Index: cc/layers/layer_impl.cc
diff --git a/cc/layers/layer_impl.cc b/cc/layers/layer_impl.cc
index e5ea46ffb0247ebe0bbcf1b0ce747138a934782f..4dc3b72c904b7a2ec84fbf36959f1be038874011 100644
--- a/cc/layers/layer_impl.cc
+++ b/cc/layers/layer_impl.cc
@@ -784,6 +784,31 @@ void LayerImpl::OnTransformIsPotentiallyAnimatingChanged(bool is_animating) {
   was_ever_ready_since_last_transform_animation_ = false;
 }
 
+void LayerImpl::OnOpacityIsCurrentlyAnimatingChanged(
+    bool is_currently_animating) {
+  DCHECK(layer_tree_impl_);
+  EffectTree& effect_tree = layer_tree_impl_->property_trees()->effect_tree;
+  EffectNode* node = effect_tree.Node(effect_tree_index());
+  if (!node)

[ajuma, 2016/05/09 16:56:55]

Here too, when does the layer have an invalid property tree index?

[jaydasika, 2016/05/10 20:22:29]

In a couple of tests. I have modified those tests to build property trees first
and removed this.

+    return;
+
+  if (node->owner_id == id())
+    node->data.is_currently_animating_opacity = is_currently_animating;
+}
+
+void LayerImpl::OnOpacityIsPotentiallyAnimatingChanged(
+    bool has_potential_animation) {
+  DCHECK(layer_tree_impl_);
+  EffectTree& effect_tree = layer_tree_impl_->property_trees()->effect_tree;
+  EffectNode* node = effect_tree.Node(effect_tree_index());
+  if (!node)
+    return;
+  if (node->owner_id == id()) {
+    node->data.has_potential_opacity_animation = has_potential_animation;
+    effect_tree.set_needs_update(true);
+  }
+}
+
 bool LayerImpl::IsActive() const {
   return layer_tree_impl_->IsActiveTree();
 }