DirectCompositorFrameSink Uses CompositorFrameSinkSupport

components/exo/compositor_frame_sink.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.h"
 
 #include "base/memory/ptr_util.h"
 #include "cc/surfaces/surface.h"
 #include "cc/surfaces/surface_manager.h"
 #include "components/exo/compositor_frame_sink_holder.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace exo {
 
 ////////////////////////////////////////////////////////////////////////////////
 // ExoComopositorFrameSink, public:
 
 CompositorFrameSink::CompositorFrameSink(const cc::FrameSinkId& frame_sink_id,
                                          cc::SurfaceManager* surface_manager,
                                          CompositorFrameSinkHolder* client)
-    : support_(this, surface_manager, frame_sink_id, nullptr, nullptr),
+    : support_(this, surface_manager, frame_sink_id, nullptr, true, true),

[Fady Samuel, 2017/02/07 17:42:18]

Please specify what these trues mean: e.g. true /*
handles_frame_sink_id_invalidation */, true /* needs_sync_points*/

[Alex Z., 2017/02/07 17:47:50]

Done.

[danakj, 2017/02/07 17:47:55]

same here

[Alex Z., 2017/02/07 17:51:33]

Done.

       client_(client) {}
 
 CompositorFrameSink::~CompositorFrameSink() {}
 
 ////////////////////////////////////////////////////////////////////////////////
 // cc::mojom::MojoCompositorFrameSink overrides:
 
 void CompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
   support_.SetNeedsBeginFrame(needs_begin_frame);
 }
@@ skipping 31 matching lines @@
 void CompositorFrameSink::ReclaimResources(
     const cc::ReturnedResourceArray& resources) {
   client_->ReclaimResources(resources);
 }
 
 void CompositorFrameSink::WillDrawSurface() {
   client_->WillDrawSurface();
 }
 
 }  // namespace exo