Change exo::SurfaceFactoryOwner to exo::ExoCompositorFrameSink

components/exo/compositor_frame_sink_holder.h

+// Copyright 2016 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 COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_
+#define COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_
+
+#include <map>
+
+#include "cc/resources/single_release_callback.h"
+#include "cc/resources/transferable_resource.h"
+#include "cc/scheduler/begin_frame_source.h"
+
+namespace exo {
+
+// TODO(staraz): Implement MojoCompositorFrameSinkClient
+class CompositorFrameSinkHolder
+    : public base::RefCounted<CompositorFrameSinkHolder> {
+ public:
+  CompositorFrameSinkHolder();
+
+  // cc::mojom::MojoCompositorFrameSinkClient:
+  void DidReceiveCompositorFrameAck();

[Fady Samuel, 2016/11/17 21:42:54]

Add implementations for these? TODOs are fine.

[Alex Z., 2016/11/23 14:25:04]

Done.

+  void OnBeginFrame(const cc::BeginFrameArgs& args);
+  void ReclaimResources(const cc::ReturnedResourceArray& resources);
+
+ private:
+  friend class base::RefCounted<CompositorFrameSinkHolder>;
+  friend class Surface;
+
+  ~CompositorFrameSinkHolder();
+
+  std::map<int,
+           std::pair<scoped_refptr<CompositorFrameSinkHolder>,
+                     std::unique_ptr<cc::SingleReleaseCallback>>>
+      release_callbacks_;
+};

[Fady Samuel, 2016/11/17 21:42:54]

nit: DISALLOW_COPY_AND_ASSIGN(CompositorFrameSinkHolder);

[Alex Z., 2016/11/23 14:25:04]

Done.

+
+}  // namespace exo
+
+#endif  // COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_