LayerTreeHost should not modify the LayerTreeSettings object. Instead it should use a temporary va…

cc/trees/layer_tree_host.h

Index: cc/trees/layer_tree_host.h
diff --git a/cc/trees/layer_tree_host.h b/cc/trees/layer_tree_host.h
index f34215fb08fa33510c23be32d9a42e6e56956b32..c8da9e3ff5a9380ce3fff2092d6a03c0d7261357 100644
--- a/cc/trees/layer_tree_host.h
+++ b/cc/trees/layer_tree_host.h
@@ -273,6 +273,7 @@ class CC_EXPORT LayerTreeHost : NON_EXPORTED_BASE(public RateLimiterClient) {
   virtual void RateLimit() OVERRIDE;
 
   bool AlwaysUsePartialTextureUpdates();
+  size_t max_partial_texture_updates() const;

[danakj, 2013/10/04 17:19:16]

Cool! Sorry to be confusing, but since this function is not just returning a
variable anymore, it should go back to its other form as
MaxPartialTextureUpdates(). Other than that, LGTM!

   bool RequestPartialTextureUpdate();
 
   void SetDeviceScaleFactor(float device_scale_factor);
@@ -386,7 +387,7 @@ class CC_EXPORT LayerTreeHost : NON_EXPORTED_BASE(public RateLimiterClient) {
   base::WeakPtr<InputHandler> input_handler_weak_ptr_;
   base::WeakPtr<TopControlsManager> top_controls_manager_weak_ptr_;
 
-  LayerTreeSettings settings_;
+  const LayerTreeSettings settings_;
   LayerTreeDebugState debug_state_;
 
   gfx::Size device_viewport_size_;