Add BeginFrameObserverProxy

content/browser/compositor/delegated_frame_host.h

Index: content/browser/compositor/delegated_frame_host.h
diff --git a/content/browser/compositor/delegated_frame_host.h b/content/browser/compositor/delegated_frame_host.h
index 42ad05d6524ebcf8e9d1105324d593bc0d341c2d..e76a8d22f55557df916d2f83769b80aa615eb420 100644
--- a/content/browser/compositor/delegated_frame_host.h
+++ b/content/browser/compositor/delegated_frame_host.h
@@ -26,6 +26,7 @@
 namespace cc {
 class SurfaceFactory;
 enum class SurfaceDrawStatus;
+struct BeginFrameArgs;
 }
 
 namespace media {
@@ -45,6 +46,8 @@ class ResizeLock;
 // display them.
 class CONTENT_EXPORT DelegatedFrameHostClient {
  public:
+  virtual ~DelegatedFrameHostClient() {}
+
   virtual ui::Layer* DelegatedFrameHostGetLayer() const = 0;
   virtual bool DelegatedFrameHostIsVisible() const = 0;
   virtual gfx::Size DelegatedFrameHostDesiredSizeInDIP() const = 0;
@@ -65,6 +68,8 @@ class CONTENT_EXPORT DelegatedFrameHostClient {
   virtual void DelegatedFrameHostUpdateVSyncParameters(
       const base::TimeTicks& timebase,
       const base::TimeDelta& interval) = 0;
+  virtual void DelegatedFrameHostSendBeginFrame(
+      const cc::BeginFrameArgs& args) = 0;
 };
 
 // The DelegatedFrameHost is used to host all of the RenderWidgetHostView state
@@ -73,6 +78,7 @@ class CONTENT_EXPORT DelegatedFrameHostClient {
 // the ui::Compositor associated with its DelegatedFrameHostClient.
 class CONTENT_EXPORT DelegatedFrameHost
     : public ui::CompositorObserver,
+      public ui::CompositorBeginFrameObserver,
       public ui::CompositorVSyncManager::Observer,
       public ui::LayerOwnerDelegate,
       public ImageTransportFactoryObserver,
@@ -81,7 +87,8 @@ class CONTENT_EXPORT DelegatedFrameHost
       public cc::SurfaceFactoryClient,
       public base::SupportsWeakPtr<DelegatedFrameHost> {
  public:
-  DelegatedFrameHost(DelegatedFrameHostClient* client);
+  DelegatedFrameHost(DelegatedFrameHostClient* client,
+                     bool begin_frame_scheduling_enabled);
   ~DelegatedFrameHost() override;
 
   bool CanCopyToBitmap() const;
@@ -113,6 +120,7 @@ class CONTENT_EXPORT DelegatedFrameHost
       scoped_ptr<RenderWidgetHostViewFrameSubscriber> subscriber);
   void EndFrameSubscription();
   bool HasFrameSubscriber() const { return frame_subscriber_; }
+  void OnSetNeedsBeginFrames(bool needs_begin_frames);
 
   // Exposed for tests.
   cc::DelegatedFrameProvider* FrameProviderForTesting() const {
@@ -147,6 +155,9 @@ class CONTENT_EXPORT DelegatedFrameHost
   void UnlockResources();
   void RequestCopyOfOutput(scoped_ptr<cc::CopyOutputRequest> request);
 
+  void StartObservingBeginFrames();
+  void StopObservingBeginFrames();
+
   // Overridden from ui::CompositorObserver:
   void OnCompositingDidCommit(ui::Compositor* compositor) override;
   void OnCompositingStarted(ui::Compositor* compositor,
@@ -160,6 +171,9 @@ class CONTENT_EXPORT DelegatedFrameHost
   void OnUpdateVSyncParameters(base::TimeTicks timebase,
                                base::TimeDelta interval) override;
 
+  // Overridden from ui::CompositorBeginFrameObserver:
+  void OnSendBeginFrame(const cc::BeginFrameArgs& args) override;
+
   // Overridden from ui::LayerOwnerObserver:
   void OnLayerRecreated(ui::Layer* old_layer, ui::Layer* new_layer) override;
 
@@ -237,11 +251,11 @@ class CONTENT_EXPORT DelegatedFrameHost
   // layer.
   bool use_surfaces_;
 
-  std::vector<base::Closure> on_compositing_did_commit_callbacks_;
-
   // The vsync manager we are observing for changes, if any.
   scoped_refptr<ui::CompositorVSyncManager> vsync_manager_;
 
+  std::vector<base::Closure> on_compositing_did_commit_callbacks_;
+
   // The current VSync timebase and interval. These are zero until the first
   // call to OnUpdateVSyncParameters().
   base::TimeTicks vsync_timebase_;
@@ -318,6 +332,17 @@ class CONTENT_EXPORT DelegatedFrameHost
       yuv_readback_pipeline_;
 
   scoped_ptr<DelegatedFrameEvictor> delegated_frame_evictor_;
+
+  // True when unified BeginFrame scheduing is used.
+  bool begin_frame_scheduling_enabled_;
+
+  // True when RenderWidget needs a BeginFrame.
+  bool needs_begin_frames_;
+
+  // Pass |last_sent_begin_frame_args_| to compositor when |this| is added as a
+  // CompositorBeginFrameObserver. With this, Compositor can determine whether
+  // latest BeginFrameArgs can be used immediately or not.
+  cc::BeginFrameArgs last_sent_begin_frame_args_;
 };
 
 }  // namespace content