Mus+Ash: Towards Unifying CompositorFrameSink terminology

content/browser/compositor/mus_browser_compositor_output_surface.h

Index: content/browser/compositor/mus_browser_compositor_output_surface.h
diff --git a/content/browser/compositor/mus_browser_compositor_output_surface.h b/content/browser/compositor/mus_browser_compositor_output_surface.h
index 2cb682bd2a154dc1a2194a168177d336d8091b33..04abb32af2d8d406ff89a85baacc8c3ccb0365df 100644
--- a/content/browser/compositor/mus_browser_compositor_output_surface.h
+++ b/content/browser/compositor/mus_browser_compositor_output_surface.h
@@ -9,6 +9,7 @@
 #include <vector>
 
 #include "base/macros.h"
+#include "cc/output/compositor_frame_sink_client.h"
 #include "content/browser/compositor/gpu_browser_compositor_output_surface.h"
 #include "gpu/command_buffer/common/mailbox.h"
 #include "gpu/ipc/common/surface_handle.h"
@@ -16,7 +17,7 @@
 
 namespace ui {
 class Window;
-class WindowSurface;
+class CompositorFrameSink;

[rjkroege, 2016/10/20 21:33:17]

alphabetical order

[Fady Samuel, 2016/10/20 23:04:10]

Done.

 }
 
 namespace content {
@@ -26,7 +27,7 @@ namespace content {
 // arriving from the GPU process.
 class MusBrowserCompositorOutputSurface
     : public GpuBrowserCompositorOutputSurface,
-      public ui::WindowSurfaceClient {
+      public cc::CompositorFrameSinkClient {
  public:
   MusBrowserCompositorOutputSurface(
       ui::Window* window,
@@ -43,9 +44,19 @@ class MusBrowserCompositorOutputSurface
   void SwapBuffers(cc::OutputSurfaceFrame frame) override;
   bool BindToClient(cc::OutputSurfaceClient* client) override;
 
-  // ui::WindowSurfaceClient:
-  void OnResourcesReturned(ui::WindowSurface* surface,
-                           const cc::ReturnedResourceArray& resources) override;
+  // ui::CompositorFrameSinkClient:

[rjkroege, 2016/10/20 21:33:17]

either it's in ui or cc. So this line is wrong?

[Fady Samuel, 2016/10/20 23:04:10]

heh, cc:: fixed.

+  void SetBeginFrameSource(cc::BeginFrameSource* source) override;
+  void ReclaimResources(const cc::ReturnedResourceArray& resources) override;
+  void SetTreeActivationCallback(const base::Closure& callback) override;
+  void DidReceiveCompositorFrameAck() override;
+  void DidLoseCompositorFrameSink() override;
+  void OnDraw(const gfx::Transform& transform,
+              const gfx::Rect& viewport,
+              bool resourceless_software_draw) override;
+  void SetMemoryPolicy(const cc::ManagedMemoryPolicy& policy) override;
+  void SetExternalTilePriorityConstraints(
+      const gfx::Rect& viewport_rect,
+      const gfx::Transform& transform) override;
 
  private:
   uint32_t AllocateResourceId();
@@ -53,7 +64,7 @@ class MusBrowserCompositorOutputSurface
   const gpu::Mailbox& GetMailboxFromResourceId(uint32_t id);
 
   ui::Window* ui_window_;
-  std::unique_ptr<ui::WindowSurface> ui_window_surface_;
+  std::unique_ptr<ui::CompositorFrameSink> ui_compositor_frame_sink_;
   std::vector<uint32_t> free_resource_ids_;
   std::vector<gpu::Mailbox> mailboxes_;