Unified BeginFrame scheduling

cc/trees/layer_tree_host.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_H_
 #define CC_TREES_LAYER_TREE_HOST_H_
 
 #include <limits>
 #include <list>
 #include <set>
@@ skipping 281 matching lines @@
   // to unregister itself.
   void InsertSwapPromiseMonitor(SwapPromiseMonitor* monitor);
   void RemoveSwapPromiseMonitor(SwapPromiseMonitor* monitor);
 
   // Call this function when you expect there to be a swap buffer.
   // See swap_promise.h for how to use SwapPromise.
   void QueueSwapPromise(scoped_ptr<SwapPromise> swap_promise);
 
   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,
                 const LayerTreeSettings& settings);
   void InitializeThreaded(
       scoped_refptr<base::SingleThreadTaskRunner> main_task_runner,
       scoped_refptr<base::SingleThreadTaskRunner> impl_task_runner);
   void InitializeSingleThreaded(
       LayerTreeHostSingleThreadClient* single_thread_client,
       scoped_refptr<base::SingleThreadTaskRunner> main_task_runner);
@@ skipping 136 matching lines @@
   scoped_refptr<Layer> outer_viewport_scroll_layer_;
 
   LayerSelectionBound selection_start_;
   LayerSelectionBound selection_end_;
 
   SharedBitmapManager* shared_bitmap_manager_;
 
   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);
 };
 
 }  // namespace cc
 
 #endif  // CC_TREES_LAYER_TREE_HOST_H_