Create RenderSurface on Effect Tree

cc/trees/layer_tree_impl.h

Index: cc/trees/layer_tree_impl.h
diff --git a/cc/trees/layer_tree_impl.h b/cc/trees/layer_tree_impl.h
index 7901c0da2b1978c38bea9cbd8150c99f2d0301ab..04e20abdedbb06f8ce3502e665bf9e6523324f76 100644
--- a/cc/trees/layer_tree_impl.h
+++ b/cc/trees/layer_tree_impl.h
@@ -232,7 +232,11 @@ class CC_EXPORT LayerTreeImpl {
     return needs_update_draw_properties_;
   }
 
-  void set_needs_full_tree_sync(bool needs) { needs_full_tree_sync_ = needs; }
+  void set_needs_full_tree_sync(bool needs) {
+    needs_full_tree_sync_ = needs;
+    if (needs)
+      property_trees_.needs_rebuild = true;

[ajuma, 2015/12/02 14:54:21]

This will only actually trigger a rebuild in tests right? (Since
CalculateDrawPropertiesAndVerify will be called with DONT_BUILD_PROPERTY_TREES
when called by LayerTreeImpl.) Please add a comment about this. Also, are there
tests that really require this? Since (I think) this code will only get reached
during a commit, won't a freshly built set of property trees get set on the
LayerTreeImpl during the commit?

[weiliangc, 2015/12/02 18:16:38]

Sorry I think I left this in during one of my attempts to make some unittests
pass. Unused and removed.

+  }
   bool needs_full_tree_sync() const { return needs_full_tree_sync_; }
 
   void ForceRedrawNextActivation() { next_activation_forces_redraw_ = true; }
@@ -391,6 +395,7 @@ class CC_EXPORT LayerTreeImpl {
   bool OpacityIsAnimatingOnImplOnly(const LayerImpl* layer) const;
   bool TransformIsAnimatingOnImplOnly(const LayerImpl* layer) const;
 
+  bool AnimationsPreserveAxisAlignment(const LayerImpl* layer) const;
   bool HasOnlyTranslationTransforms(const LayerImpl* layer) const;
 
   bool MaximumTargetScale(const LayerImpl* layer, float* max_scale) const;