CC: Add a scheduled action for ManageTiles

cc/trees/layer_tree_host_impl.h

Index: cc/trees/layer_tree_host_impl.h
diff --git a/cc/trees/layer_tree_host_impl.h b/cc/trees/layer_tree_host_impl.h
index 488cc3688692a4fb8cdc4e3c398e6149422bc7b6..ce1b9a83702127167f4ed26f3a85f8fca567faa3 100644
--- a/cc/trees/layer_tree_host_impl.h
+++ b/cc/trees/layer_tree_host_impl.h
@@ -63,6 +63,7 @@ class LayerTreeHostImplClient {
   virtual void SetNeedsRedrawRectOnImplThread(gfx::Rect damage_rect) = 0;
   virtual void DidInitializeVisibleTileOnImplThread() = 0;
   virtual void SetNeedsCommitOnImplThread() = 0;
+  virtual void SetNeedsManageTilesOnImplThread() = 0;
   virtual void PostAnimationEventsToMainThreadOnImplThread(
       scoped_ptr<AnimationEventsVector> events,
       base::Time wall_clock_time) = 0;
@@ -252,7 +253,7 @@ class CC_EXPORT LayerTreeHostImpl
 
   virtual bool SwapBuffers(const FrameData& frame);
   void SetNeedsBeginFrame(bool enable);
-  void SetNeedsManageTiles() { manage_tiles_needed_ = true; }
+  void WillModifyTilePriorities();
 
   void Readback(void* pixels, gfx::Rect rect_in_device_viewport);
 
@@ -425,7 +426,8 @@ class CC_EXPORT LayerTreeHostImpl
     return animation_registrar_->active_animation_controllers();
   }
 
-  bool manage_tiles_needed() const { return manage_tiles_needed_; }
+  bool tile_priorities_dirty() const { return tile_priorities_dirty_; }
+  bool manage_tiles_needed() const { return tile_priorities_dirty_; }
 
   LayerTreeHostImplClient* client_;
   Proxy* proxy_;
@@ -507,7 +509,7 @@ class CC_EXPORT LayerTreeHostImpl
   bool should_bubble_scrolls_;
   bool wheel_scrolling_;
 
-  bool manage_tiles_needed_;
+  bool tile_priorities_dirty_;

[reveman, 2013/09/11 15:27:45]

why do we still need this state here? isn't it enough with the scheduler state?

[epennerAtGoogle, 2013/09/11 18:43:00]

We still call ManageTiles explicitly in a few places (see OnTimerTick() and
CommitComplete()). The current scheduler action only replaces the ManageTiles()
'semi-regular' ManageTiles that used to happen during draws. To remove the dirty
flag we would need to remove the other cases as well. If it's clear I can try
that, but otherwise I'd prefer that happens in another patch.

 
   // The optional delegate for the root layer scroll offset.
   LayerScrollOffsetDelegate* root_layer_scroll_offset_delegate_;