cc: Move main thread hierarchy dependencies into PropertyTreeBuilder

cc/trees/draw_property_utils.cc

 // Copyright 2014 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.
 
 #include "cc/trees/draw_property_utils.h"
 
 #include <stddef.h>
 
 #include <vector>
 
@@ skipping 53 matching lines @@
   DCHECK(layer->background_filters().IsEmpty()) << "layer: " << layer->id();
   DCHECK(!IsRootLayer(layer)) << "layer: " << layer->id();
   if (EffectNodeOwner(layer)->replica_layer() == layer)
     return;
   DCHECK(!layer->mask_layer()) << "layer: " << layer->id();
   DCHECK(!layer->replica_layer()) << "layer: " << layer->id();
   DCHECK(!layer->HasCopyRequest()) << "layer: " << layer->id();
   ValidateIsNotRootForIsolatedGroup(layer);
 }
 
-static void ValidateRenderSurfacesRecursive(Layer* layer) {
-  ValidateRenderSurfaceForLayer(layer);
-  for (size_t i = 0; i < layer->children().size(); ++i)
-    ValidateRenderSurfacesRecursive(layer->child_at(i));
+static void ValidateRenderSurfacesRecursive(Layer* root_layer) {
+  for (auto* layer : *root_layer->layer_tree_host())
+    ValidateRenderSurfaceForLayer(layer);
 }
 #endif
 
 template <typename LayerType>
 void CalculateVisibleRects(
     const typename LayerType::LayerListType& visible_layer_list,
     const ClipTree& clip_tree,
     const TransformTree& transform_tree,
     bool non_root_surfaces_enabled) {
   for (auto& layer : visible_layer_list) {
@@ skipping 279 matching lines @@
 
   EffectNode* node = effect_tree->Node(layer->effect_tree_index());
 
   if (node->owner_id == layer->id() && node->data.has_render_surface)
     layer->SetHasRenderSurface(true);
   else
     layer->SetHasRenderSurface(false);
 }
 
 void UpdateRenderSurfacesForLayersRecursive(EffectTree* effect_tree,
-                                            Layer* layer) {
-  UpdateRenderSurfaceForLayer(effect_tree, true, layer);
-
-  for (size_t i = 0; i < layer->children().size(); ++i) {
-    UpdateRenderSurfacesForLayersRecursive(effect_tree, layer->child_at(i));
-  }
+                                            Layer* root_layer) {
+  for (auto* layer : *root_layer->layer_tree_host())
+    UpdateRenderSurfaceForLayer(effect_tree, true, layer);

[jaydasika, 2016/04/27 18:21:10]

This and ValidateRenderSurfacesRecursive can be deleted and we can both update
and validate in a single iteration that can go into the caller :
UpdateRenderSurfaces. (ComputeVisibleRects does the same thing for impl-side)

 }
 
 }  // namespace
 
 template <typename LayerType>
 static inline bool LayerShouldBeSkippedInternal(
     LayerType* layer,
     bool layer_is_drawn,
     const TransformTree& transform_tree,
     const EffectTree& effect_tree) {
@@ skipping 733 matching lines @@
 void UpdateElasticOverscroll(PropertyTrees* property_trees,
                              const Layer* overscroll_elasticity_layer,
                              const gfx::Vector2dF& elastic_overscroll) {
   UpdateElasticOverscrollInternal(property_trees, overscroll_elasticity_layer,
                                   elastic_overscroll);
 }
 
 }  // namespace draw_property_utils
 
 }  // namespace cc