Create RenderSurface on Effect Tree

cc/trees/layer_tree_impl.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_TREES_LAYER_TREE_IMPL_H_
 #define CC_TREES_LAYER_TREE_IMPL_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 214 matching lines @@
   void BuildPropertyTreesForTesting();
   void IncrementRenderSurfaceListIdForTesting();
 
   void set_needs_update_draw_properties() {
     needs_update_draw_properties_ = true;
   }
   bool needs_update_draw_properties() const {
     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; }
 
   void set_has_ever_been_drawn(bool has_drawn) {
     has_ever_been_drawn_ = has_drawn;
   }
   bool has_ever_been_drawn() const { return has_ever_been_drawn_; }
 
   void set_ui_resource_request_queue(const UIResourceRequestQueue& queue);
@@ skipping 138 matching lines @@
   bool HasPotentiallyRunningTransformAnimation(const LayerImpl* layer) const;
 
   bool HasAnyAnimationTargetingProperty(
       const LayerImpl* layer,
       Animation::TargetProperty property) const;
 
   bool FilterIsAnimatingOnImplOnly(const LayerImpl* layer) const;
   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;
   bool AnimationStartScale(const LayerImpl* layer, float* start_scale) const;
 
   bool HasFilterAnimationThatInflatesBounds(const LayerImpl* layer) const;
   bool HasTransformAnimationThatInflatesBounds(const LayerImpl* layer) const;
   bool HasAnimationThatInflatesBounds(const LayerImpl* layer) const;
 
   bool FilterAnimationBoundsForBox(const LayerImpl* layer,
@@ skipping 97 matching lines @@
 
   scoped_ptr<PendingPageScaleAnimation> pending_page_scale_animation_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(LayerTreeImpl);
 };
 
 }  // namespace cc
 
 #endif  // CC_TREES_LAYER_TREE_IMPL_H_