cc : Stop pushing properties not used by LayerImpl to LayerImpl. (2)

cc/layers/layer.h

Index: cc/layers/layer.h
diff --git a/cc/layers/layer.h b/cc/layers/layer.h
index 220a9dba884eacf73e804312c111d895985887b6..5c5d52d605af2767b797a09ac816821783c73857 100644
--- a/cc/layers/layer.h
+++ b/cc/layers/layer.h
@@ -487,6 +487,9 @@ class CC_EXPORT Layer : public base::RefCounted<Layer> {
   void SetSubtreePropertyChanged();
   bool subtree_property_changed() const { return subtree_property_changed_; }
 
+  void SetLayerPropertyChanged();
+  bool layer_property_changed() const { return layer_property_changed_; }

[ajuma, 2016/04/21 23:59:14]

There aren't any real non-test callers of SetForceRenderSurface. It's exposed on
WebLayer and ui::Layer but not used in a non-trivial way (ui::Layer uses it in a
test, but that can be deleted). What if we remove
WebLayer::setForceRenderSurface and ui::Layer::SetForceRenderSurface, and rename
the cc::Layer method to SetForRenderSurfaceForTesting? Then we don't need to add
or push this layer_property_changed_ bool (since any tests that depend on
SetForceRenderSurface to cause damage should be changed to not depend on this).
Does that seem reasonable?

[jaydasika, 2016/04/22 17:00:38]

Done.

+
   void DidBeginTracing();
 
   int num_copy_requests_in_target_subtree();
@@ -643,6 +646,7 @@ class CC_EXPORT Layer : public base::RefCounted<Layer> {
   bool transform_is_invertible_ : 1;
   bool has_render_surface_ : 1;
   bool subtree_property_changed_ : 1;
+  bool layer_property_changed_ : 1;
   Region non_fast_scrollable_region_;
   Region touch_event_handler_region_;
   gfx::PointF position_;