Make RendererCompositorFrameSink derive from ClientCompositorFrameSink

content/renderer/gpu/renderer_compositor_frame_sink.h

Index: content/renderer/gpu/renderer_compositor_frame_sink.h
diff --git a/content/renderer/gpu/renderer_compositor_frame_sink.h b/content/renderer/gpu/renderer_compositor_frame_sink.h
index e80c047dcc334d3d1593b7ad15be10204c265237..29d14de95e2e6c6ab9a8014bdd79122081332d07 100644
--- a/content/renderer/gpu/renderer_compositor_frame_sink.h
+++ b/content/renderer/gpu/renderer_compositor_frame_sink.h
@@ -28,6 +28,7 @@
 #include "content/renderer/gpu/compositor_forwarding_message_filter.h"
 #include "ipc/ipc_sync_message_filter.h"
 #include "mojo/public/cpp/bindings/binding.h"
+#include "services/ui/public/cpp/client_compositor_frame_sink.h"
 #include "ui/gfx/selection_bound.h"
 
 namespace IPC {
@@ -45,10 +46,7 @@ class FrameSwapMessageQueue;
 // This class can be created only on the main thread, but then becomes pinned
 // to a fixed thread when BindToClient is called.
 class RendererCompositorFrameSink
-    : NON_EXPORTED_BASE(public cc::CompositorFrameSink),
-      NON_EXPORTED_BASE(public base::NonThreadSafe),
-      NON_EXPORTED_BASE(public cc::mojom::MojoCompositorFrameSinkClient),
-      public cc::ExternalBeginFrameSourceClient {
+    : NON_EXPORTED_BASE(public ui::ClientCompositorFrameSink) {
  public:
   RendererCompositorFrameSink(
       int32_t routing_id,
@@ -71,11 +69,10 @@ class RendererCompositorFrameSink
       scoped_refptr<FrameSwapMessageQueue> swap_frame_message_queue);
   ~RendererCompositorFrameSink() override;
 
-  // cc::CompositorFrameSink implementation.
+  // Overriden from ui::ClientCompositorFrameSink.
   bool BindToClient(cc::CompositorFrameSinkClient* client) override;
   void DetachFromClient() override;
   void SubmitCompositorFrame(cc::CompositorFrame frame) override;
-  void DidNotProduceFrame(const cc::BeginFrameAck& ack) override;
 
  private:
   class RendererCompositorFrameSinkProxy
@@ -101,14 +98,8 @@ class RendererCompositorFrameSink
   void OnMessageReceived(const IPC::Message& message);
   void OnBeginFrameIPC(const cc::BeginFrameArgs& args);
 
-  // cc::mojom::MojoCompositorFrameSinkClient implementation.
-  void DidReceiveCompositorFrameAck(
-      const cc::ReturnedResourceArray& resources) override;
+  // Overriden from ui::ClientCompositorFrameSink.
   void OnBeginFrame(const cc::BeginFrameArgs& args) override;
-  void ReclaimResources(const cc::ReturnedResourceArray& resources) override;
-
-  // cc::ExternalBeginFrameSourceClient implementation.
-  void OnNeedsBeginFrames(bool need_begin_frames) override;
 
   scoped_refptr<CompositorForwardingMessageFilter>
       compositor_frame_sink_filter_;
@@ -117,16 +108,12 @@ class RendererCompositorFrameSink
   scoped_refptr<RendererCompositorFrameSinkProxy> compositor_frame_sink_proxy_;
   scoped_refptr<IPC::SyncMessageFilter> message_sender_;
   scoped_refptr<FrameSwapMessageQueue> frame_swap_message_queue_;
-  std::unique_ptr<cc::SyntheticBeginFrameSource> synthetic_begin_frame_source_;
-  std::unique_ptr<cc::ExternalBeginFrameSource> external_begin_frame_source_;
   int routing_id_;
 
   cc::LocalSurfaceId local_surface_id_;
   cc::LocalSurfaceIdAllocator id_allocator_;
   RenderWidgetSurfaceProperties current_surface_properties_;
 
-  base::ThreadChecker thread_checker_;
-
   cc::mojom::MojoCompositorFrameSinkPtr sink_;
   cc::mojom::MojoCompositorFrameSinkPtrInfo sink_info_;
   cc::mojom::MojoCompositorFrameSinkClientRequest sink_client_request_;