cc: split UpdateGpuRasterizationStatus() into two parts.

cc/trees/layer_tree_host_impl.h

 // 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.
 
 #ifndef CC_TREES_LAYER_TREE_HOST_IMPL_H_
 #define CC_TREES_LAYER_TREE_HOST_IMPL_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 303 matching lines @@
   OutputSurface* output_surface() const { return output_surface_.get(); }
 
   std::string LayerTreeAsJson() const;
 
   void FinishAllRendering();
   int SourceAnimationFrameNumber() const;
 
   virtual bool InitializeRenderer(scoped_ptr<OutputSurface> output_surface);
   TileManager* tile_manager() { return tile_manager_.get(); }
 
   void set_has_gpu_rasterization_trigger(bool flag) {

[Stephen White, 2015/05/13 19:44:17]

This should probably be CamelCased, now that it's no longer just a setter.

[danakj, 2015/05/13 19:50:58]

this isn't a simple setter anymore, no hacker case

[Stephen White, 2015/05/13 20:15:20]

Done.

     has_gpu_rasterization_trigger_ = flag;
+    UpdateGpuRasterizationStatus();
   }
   void set_content_is_suitable_for_gpu_rasterization(bool flag) {

[Stephen White, 2015/05/13 19:44:17]

Same here.

[danakj, 2015/05/13 19:50:58]

dittos

[Stephen White, 2015/05/13 20:15:20]

Done.

     content_is_suitable_for_gpu_rasterization_ = flag;
+    UpdateGpuRasterizationStatus();
   }
-  void UpdateGpuRasterizationStatus();
+  void UpdateTreeResourcesIfNeeded();

[danakj, 2015/05/13 19:50:58]

UpdateTreeResourcesForGpuRasterizationIfNeeded?

[Stephen White, 2015/05/13 20:15:20]

Done.

   bool use_gpu_rasterization() const { return use_gpu_rasterization_; }
   bool use_msaa() const { return use_msaa_; }
 
   GpuRasterizationStatus gpu_rasterization_status() const {
     return gpu_rasterization_status_;
   }
 
   bool create_low_res_tiling() const {
     return settings_.create_low_res_tiling && !use_gpu_rasterization_;
   }
@@ skipping 223 matching lines @@
       LayerImpl* layer_impl,
       const gfx::PointF& viewport_point,
       const gfx::Vector2dF& viewport_delta);
 
   void CreateAndSetRenderer();
   void CreateAndSetTileManager();
   void DestroyTileManager();
   void ReleaseTreeResources();
   void RecreateTreeResources();
 
+  void UpdateGpuRasterizationStatus();
+
   bool IsSynchronousSingleThreaded() const;
 
   Viewport* viewport() { return viewport_.get(); }
 
   // Scroll by preferring to move the outer viewport first, only moving the
   // inner if the outer is at its scroll extents.
   void ScrollViewportBy(gfx::Vector2dF scroll_delta);
   // Scroll by preferring to move the inner viewport first, only moving the
   // outer if the inner is at its scroll extents.
   void ScrollViewportInnerFirst(gfx::Vector2dF scroll_delta);
@@ skipping 55 matching lines @@
   // |resource_provider_| and |tile_manager_| can be NULL, e.g. when using tile-
   // free rendering - see OutputSurface::ForcedDrawToSoftwareDevice().
   // |tile_manager_| can also be NULL when raster_enabled is false.
   scoped_ptr<ResourceProvider> resource_provider_;
   scoped_ptr<TileManager> tile_manager_;
   bool content_is_suitable_for_gpu_rasterization_;
   bool has_gpu_rasterization_trigger_;
   bool use_gpu_rasterization_;
   bool use_msaa_;
   GpuRasterizationStatus gpu_rasterization_status_;
+  bool tree_resources_dirty_;

[danakj, 2015/05/13 19:50:58]

_for_gpu_rasterization_

[Stephen White, 2015/05/13 20:15:20]

Done.

   scoped_ptr<TileTaskWorkerPool> tile_task_worker_pool_;
   scoped_ptr<ResourcePool> resource_pool_;
   scoped_ptr<ResourcePool> staging_resource_pool_;
   scoped_ptr<Renderer> renderer_;
 
   GlobalStateThatImpactsTilePriority global_tile_state_;
 
   // Tree currently being drawn.
   scoped_ptr<LayerTreeImpl> active_tree_;
 
@@ skipping 101 matching lines @@
   scoped_ptr<FrameTimingTracker> frame_timing_tracker_;
 
   scoped_ptr<Viewport> viewport_;
 
   DISALLOW_COPY_AND_ASSIGN(LayerTreeHostImpl);
 };
 
 }  // namespace cc
 
 #endif  // CC_TREES_LAYER_TREE_HOST_IMPL_H_