cc: Move tiling management out of draw properties calculation.

cc/trees/layer_tree_impl.cc

 // Copyright 2011 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/layer_tree_impl.h"
 
 #include "base/debug/trace_event.h"
 #include "cc/animation/keyframed_animation_curve.h"
 #include "cc/animation/scrollbar_animation_controller.h"
 #include "cc/animation/scrollbar_animation_controller_linear_fade.h"
@@ skipping 492 matching lines @@
     // LayerIterator is used here instead of CallFunctionForSubtree to only
     // UpdateTilePriorities on layers that will be visible (and thus have valid
     // draw properties) and not because any ordering is required.
     typedef LayerIterator<LayerImpl> LayerIteratorType;
     LayerIteratorType end = LayerIteratorType::End(&render_surface_layer_list_);
     for (LayerIteratorType it =
              LayerIteratorType::Begin(&render_surface_layer_list_);
          it != end;
          ++it) {
       LayerImpl* layer = *it;
+

[sohanjg, 2014/05/13 15:56:59]

will this be the best place to managetiling ?

[enne (OOO), 2014/05/13 18:04:09]

No.  I think UpdateTilePriorities should call ManageTilings itself.

+      // Untie ManageTilings from CalculateDrawProperties
+      layer->ManageTilings(layer->TransformIsAnimating(), 1.f);
+
       if (it.represents_itself())
         layer->UpdateTilePriorities();
 
       if (!it.represents_contributing_render_surface())
         continue;
 
       if (layer->mask_layer())
         layer->mask_layer()->UpdateTilePriorities();
       if (layer->replica_layer() && layer->replica_layer()->mask_layer())
         layer->replica_layer()->mask_layer()->UpdateTilePriorities();
@@ skipping 493 matching lines @@
   current->ReleaseResources();
   if (current->mask_layer())
     ReleaseResourcesRecursive(current->mask_layer());
   if (current->replica_layer())
     ReleaseResourcesRecursive(current->replica_layer());
   for (size_t i = 0; i < current->children().size(); ++i)
     ReleaseResourcesRecursive(current->children()[i]);
 }
 
 }  // namespace cc