Add CompositorFrameSinkClientBinding To Be Used By FrameGenerator

services/ui/ws/compositor_frame_sink_client_binding.h

+// Copyright 2017 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 SERVICES_UI_WS_COMPOSITOR_FRAME_SINK_CLIENT_BINDING_H_
+#define SERVICES_UI_WS_COMPOSITOR_FRAME_SINK_CLIENT_BINDING_H_
+
+#include "base/macros.h"
+#include "cc/ipc/frame_sink_manager.mojom.h"
+#include "cc/ipc/mojo_compositor_frame_sink.mojom.h"
+#include "cc/surfaces/local_surface_id_allocator.h"
+#include "mojo/public/cpp/bindings/binding.h"
+
+namespace ui {
+namespace ws {
+
+// CompositorFrameSinkClientBinding manages the binding between a FrameGenerator
+// and its MojoCompositorFrameSink. CompositorFrameSinkClientBinding exists so
+// that a mock implementation of MojoCompositorFrameSink can be injected for
+// tests. FrameGenerator owns its associated CompositorFrameSinkClientBinding.
+class CompositorFrameSinkClientBinding

[Fady Samuel, 2017/05/19 11:51:03]

Ahh this is pretty cool. Maybe we don't need this at all!

Maybe FrameGenerator can take in std::unique_ptr<MojoCompositorFrameSink>
directly instead of this CompositorFrameSinkClientBinding.

[Alex Z., 2017/05/19 13:26:52]

Done.

+    : public cc::mojom::MojoCompositorFrameSink {
+ public:
+  ~CompositorFrameSinkClientBinding() override = default;
+};
+
+// Bindings implementation of CompositorFrameSinkClientBinding.
+class DefaultCompositorFrameSinkClientBinding
+    : public CompositorFrameSinkClientBinding {
+ public:
+  DefaultCompositorFrameSinkClientBinding(
+      cc::mojom::MojoCompositorFrameSinkClient* sink_client,
+      cc::mojom::MojoCompositorFrameSinkClientRequest sink_client_request,
+      cc::mojom::MojoCompositorFrameSinkAssociatedPtr compositor_frame_sink,
+      cc::mojom::DisplayPrivateAssociatedPtr display_private);
+  ~DefaultCompositorFrameSinkClientBinding() override;
+
+ private:
+  // CompositorFrameSinkClientBinding implementation:
+  void SubmitCompositorFrame(const cc::LocalSurfaceId& local_surface_id,
+                             cc::CompositorFrame frame) override;
+  void SetNeedsBeginFrame(bool needs_begin_frame) override;
+  void BeginFrameDidNotSwap(const cc::BeginFrameAck& ack) override;
+  void EvictCurrentSurface() override;
+
+  mojo::Binding<cc::mojom::MojoCompositorFrameSinkClient> binding_;
+  cc::mojom::DisplayPrivateAssociatedPtr display_private_;
+  cc::mojom::MojoCompositorFrameSinkAssociatedPtr compositor_frame_sink_;
+  cc::LocalSurfaceId local_surface_id_;
+  cc::LocalSurfaceIdAllocator id_allocator_;
+  gfx::Size last_submitted_frame_size_;
+
+  DISALLOW_COPY_AND_ASSIGN(DefaultCompositorFrameSinkClientBinding);
+};
+}
+}
+
+#endif  // SERVICES_UI_WS_COMPOSITOR_FRAME_SINK_CLIENT_BINDING_H_