Switching to CompositorFrameSinkSupport in android_webview::SurfacesInstance

cc/surfaces/compositor_frame_sink_support_client.h

Index: cc/surfaces/compositor_frame_sink_support_client.h
diff --git a/cc/surfaces/compositor_frame_sink_support_client.h b/cc/surfaces/compositor_frame_sink_support_client.h
index e3f6dab23e141af7ecbce53857713369cd04b3dc..bc612ff8e8daf8c9317d792874b76555ba9c9600 100644
--- a/cc/surfaces/compositor_frame_sink_support_client.h
+++ b/cc/surfaces/compositor_frame_sink_support_client.h
@@ -6,12 +6,13 @@
 #define CC_SURFACES_COMPOSITOR_FRAME_SINK_SUPPORT_CLIENT_H_
 
 #include "cc/resources/returned_resource.h"
+#include "cc/surfaces/display_client.h"
 
 namespace cc {
 
 struct BeginFrameArgs;
 
-class CompositorFrameSinkSupportClient {
+class CompositorFrameSinkSupportClient : public DisplayClient {

[danakj, 2017/01/24 17:12:33]

This is not what I would expect, I don't think it's common for a client to be
another client, and it seems like it can impose architectural limitations on our
code that are not required.

[Saman Sami, 2017/01/24 17:25:09]

Done.

  public:
   // Notification that the previous CompositorFrame given to
   // SubmitCompositorFrame() has been processed and that another frame
@@ -32,7 +33,7 @@ class CompositorFrameSinkSupportClient {
   virtual void WillDrawSurface() = 0;
 
  protected:
-  virtual ~CompositorFrameSinkSupportClient() {}
+  ~CompositorFrameSinkSupportClient() override {}
 };
 
 }  // namespace cc