Unified BeginFrame scheduling

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 2fa5b5880831ba9610aa7f35af5019d232ec4a0b..adc1625e902af1976787ce1d3881561a5b2952c4 100644
--- a/cc/trees/layer_tree_host.h
+++ b/cc/trees/layer_tree_host.h
@@ -299,6 +299,10 @@ class CC_EXPORT LayerTreeHost {
 
   void BreakSwapPromises(SwapPromise::DidNotSwapReason reason);
 
+  void SendBeginFrame(const BeginFrameArgs& args);

[brianderson, 2014/08/21 00:13:59]

SendBeginFrameToChildren?

[simonhong, 2014/08/26 08:24:47]

Removed this and RequestBeginFrame().
Changed Scheduler as a subclass of BeginFrameManager::Delegate.

+
+  void RequestBeginFrame();
+
  protected:
   LayerTreeHost(LayerTreeHostClient* client,
                 SharedBitmapManager* manager,
@@ -455,6 +459,9 @@ class CC_EXPORT LayerTreeHost {
   ScopedPtrVector<SwapPromise> swap_promise_list_;
   std::set<SwapPromiseMonitor*> swap_promise_monitor_;
 
+  // true if new BeginFrame is requested from compositor.
+  bool begin_frame_requested_;

[brianderson, 2014/08/21 00:13:59]

I don't think LTH needs to store this. If early outs are needed, it should be
done at a higher level.

[simonhong, 2014/08/26 08:24:48]

Removed.

+
   DISALLOW_COPY_AND_ASSIGN(LayerTreeHost);
 };