cc: Compute SurfacePropertyChanged without depending on owning layer

cc/trees/property_tree_builder.cc

Index: cc/trees/property_tree_builder.cc
diff --git a/cc/trees/property_tree_builder.cc b/cc/trees/property_tree_builder.cc
index 4a85210b446311b33752dd2c2b6707dcd179af5f..5d11dbf3c69f58be553d55324eab1cfe3021e2ea 100644
--- a/cc/trees/property_tree_builder.cc
+++ b/cc/trees/property_tree_builder.cc
@@ -849,6 +849,14 @@ static inline bool HasCopyRequest(LayerImpl* layer) {
   return !layer->test_properties()->copy_requests.empty();
 }
 
+static inline bool PropertyChanged(Layer* layer) {
+  return layer->layer_property_changed() || layer->subtree_property_changed();
+}
+
+static inline bool PropertyChanged(LayerImpl* layer) {
+  return false;
+}
+
 template <typename LayerType>
 bool ShouldCreateRenderSurface(LayerType* layer,
                                gfx::Transform current_transform,
@@ -1025,6 +1033,7 @@ bool AddEffectNodeIfNeeded(
   node.subtree_hidden = HideLayerAndSubtree(layer);
   node.is_currently_animating_opacity = OpacityIsAnimating(layer);
   node.is_currently_animating_filter = FilterIsAnimating(layer);
+  node.effect_changed = PropertyChanged(layer);

[jaydasika, 2016/09/23 18:18:19]

I think this change has the side effect that we now damage the entire subtree
even for layer->property_changed() && !subtree_property_changed(). So, we don't
need Layer::layer_property_changed_ anymore.

[ajuma, 2016/09/23 22:31:12]

Done.

 
   EffectTree& effect_tree = data_for_children->property_trees->effect_tree;
   if (MaskLayer(layer)) {