cc : Stop calculating visible rects on main thread

cc/trees/draw_property_utils.h

 // 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.
 
 #ifndef CC_TREES_DRAW_PROPERTY_UTILS_H_
 #define CC_TREES_DRAW_PROPERTY_UTILS_H_
 
 #include "cc/base/cc_export.h"
 #include "cc/layers/layer_collections.h"
 
@@ skipping 36 matching lines @@
     Layer* root_layer,
     const Layer* page_scale_layer,
     const Layer* inner_viewport_scroll_layer,
     const Layer* outer_viewport_scroll_layer,
     const Layer* overscroll_elasticity_layer,
     const gfx::Vector2dF& elastic_overscroll,
     float page_scale_factor,
     float device_scale_factor,
     const gfx::Rect& viewport,
     const gfx::Transform& device_transform,
-    bool can_render_to_separate_surface,
-    PropertyTrees* property_trees,
-    LayerList* update_layer_list);
+    PropertyTrees* property_trees);
 
 void CC_EXPORT BuildPropertyTreesAndComputeVisibleRects(
     LayerImpl* root_layer,
     const LayerImpl* page_scale_layer,
     const LayerImpl* inner_viewport_scroll_layer,
     const LayerImpl* outer_viewport_scroll_layer,
     const LayerImpl* overscroll_elasticity_layer,
     const gfx::Vector2dF& elastic_overscroll,
     float page_scale_factor,
     float device_scale_factor,
     const gfx::Rect& viewport,
     const gfx::Transform& device_transform,
     bool can_render_to_separate_surface,
     PropertyTrees* property_trees,
     LayerImplList* visible_layer_list);
 
-void CC_EXPORT ComputeVisibleRects(Layer* root_layer,
-                                   PropertyTrees* property_trees,
-                                   bool can_render_to_separate_surface,
-                                   LayerList* update_layer_list);
+void CC_EXPORT Update(Layer* root_layer,

[jaydasika, 2016/04/02 02:54:10]

need to name this better :) It updates property trees, render surfaces and finds
layers that need update.

[ajuma, 2016/04/04 14:15:05]

How about splitting this into three functions?

[jaydasika, 2016/04/04 18:41:09]

Done.

+                      PropertyTrees* property_trees,
+                      bool can_render_to_separate_surface,
+                      LayerList* update_layer_list);
+
+void CC_EXPORT
+ComputeVisibleRectsForTesting(PropertyTrees* property_trees,
+                              bool can_render_to_separate_surface,
+                              LayerList* visible_layer_list);
 
 void CC_EXPORT ComputeVisibleRects(LayerImpl* root_layer,
                                    PropertyTrees* property_trees,
                                    bool can_render_to_separate_surface,
                                    LayerImplList* visible_layer_list);
 
 void CC_EXPORT ComputeLayerDrawProperties(LayerImpl* layer,
                                           const PropertyTrees* property_trees,
                                           bool layers_always_allowed_lcd_text,
                                           bool can_use_lcd_text);
@@ skipping 32 matching lines @@
 
 void CC_EXPORT
 UpdateElasticOverscroll(PropertyTrees* property_trees,
                         const Layer* overscroll_elasticity_layer,
                         const gfx::Vector2dF& elastic_overscroll);
 
 }  // namespace draw_property_utils
 }  // namespace cc
 
 #endif  // CC_TREES_DRAW_PROPERTY_UTILS_H_