Implement aura::Window::CreateCompositorFrameSink()

components/exo/compositor_frame_sink_holder.cc

 // 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.
 
 #include "components/exo/compositor_frame_sink_holder.h"
 
+#include "cc/output/compositor_frame_sink.h"
 #include "cc/resources/returned_resource.h"
 #include "components/exo/surface.h"
 
 namespace exo {
 
 ////////////////////////////////////////////////////////////////////////////////
 // CompositorFrameSinkHolder, public:
 
 CompositorFrameSinkHolder::CompositorFrameSinkHolder(
     Surface* surface,
-    const cc::FrameSinkId& frame_sink_id,
-    cc::SurfaceManager* surface_manager)
+    std::unique_ptr<cc::CompositorFrameSink> frame_sink)
     : surface_(surface),
-      frame_sink_(
-          new CompositorFrameSink(frame_sink_id, surface_manager, this)),
-      begin_frame_source_(base::MakeUnique<cc::ExternalBeginFrameSource>(this)),
+      frame_sink_(std::move(frame_sink)),
       weak_factory_(this) {
   surface_->AddSurfaceObserver(this);
-  surface_->SetBeginFrameSource(begin_frame_source_.get());
+  frame_sink_->BindToClient(this);
+}
+
+CompositorFrameSinkHolder::~CompositorFrameSinkHolder() {
+  frame_sink_->DetachFromClient();
+  if (surface_)
+    surface_->RemoveSurfaceObserver(this);
+
+  // Release all resources which aren't returned from CompositorFrameSink.
+  const gpu::SyncToken token;

[reveman, 2017/05/09 17:35:58]

nit: how common is it that we actually have toms release callbacks here? If we
do it's most likely one so can we remove this local variable and just do
"Run(gpu::SyncToken(), false)" below instead?

[Peng, 2017/05/09 19:48:49]

I tested with several wayland clients. the size of release_callbacks_ is one. So
removed the local variable. Done

+  for (auto& callback : release_callbacks_)
+    callback.second.Run(token, false);
 }
 
 bool CompositorFrameSinkHolder::HasReleaseCallbackForResource(
     cc::ResourceId id) {
   return release_callbacks_.find(id) != release_callbacks_.end();
 }
 
 void CompositorFrameSinkHolder::SetResourceReleaseCallback(
     cc::ResourceId id,
     const cc::ReleaseCallback& callback) {
   DCHECK(!callback.is_null());
   release_callbacks_[id] = callback;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
-// cc::mojom::MojoCompositorFrameSinkClient overrides:
+// cc::CompositorFrameSinkClient overrides:
 
-void CompositorFrameSinkHolder::DidReceiveCompositorFrameAck(
-    const cc::ReturnedResourceArray& resources) {
-  ReclaimResources(resources);
+void CompositorFrameSinkHolder::SetBeginFrameSource(
+    cc::BeginFrameSource* source) {
   if (surface_)
-    surface_->DidReceiveCompositorFrameAck();
-}
-
-void CompositorFrameSinkHolder::OnBeginFrame(const cc::BeginFrameArgs& args) {
-  begin_frame_source_->OnBeginFrame(args);
+    surface_->SetBeginFrameSource(source);
 }
 
 void CompositorFrameSinkHolder::ReclaimResources(
     const cc::ReturnedResourceArray& resources) {
   for (auto& resource : resources) {
     auto it = release_callbacks_.find(resource.id);
     DCHECK(it != release_callbacks_.end());
     if (it != release_callbacks_.end()) {
       it->second.Run(resource.sync_token, resource.lost);
       release_callbacks_.erase(it);
     }
   }
 }
 
-////////////////////////////////////////////////////////////////////////////////
-// cc::ExternalBeginFrameSourceClient overrides:
-
-void CompositorFrameSinkHolder::OnNeedsBeginFrames(bool needs_begin_frames) {
-  frame_sink_->SetNeedsBeginFrame(needs_begin_frames);
-}
-
-void CompositorFrameSinkHolder::OnDidFinishFrame(const cc::BeginFrameAck& ack) {
-  // If there was damage, the submitted CompositorFrame includes the ack.
-  if (!ack.has_damage)
-    frame_sink_->BeginFrameDidNotSwap(ack);
+void CompositorFrameSinkHolder::DidReceiveCompositorFrameAck() {
+  if (surface_)
+    surface_->DidReceiveCompositorFrameAck();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // SurfaceObserver overrides:
 
 void CompositorFrameSinkHolder::OnSurfaceDestroying(Surface* surface) {
   surface_->RemoveSurfaceObserver(this);
   surface_ = nullptr;
 }
 
-////////////////////////////////////////////////////////////////////////////////
-// ExoComopositorFrameSink, private:
-
-CompositorFrameSinkHolder::~CompositorFrameSinkHolder() {
-  if (surface_)
-    surface_->RemoveSurfaceObserver(this);
-}
-
 }  // namespace exo