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),
       client_(client) {}
 
-CompositorFrameSink::~CompositorFrameSink() {}
+CompositorFrameSink::~CompositorFrameSink() {
+  support_.InvalidateFrameSinkId();

[Fady Samuel, 2017/01/23 21:56:12]

Why is this necessary? support should do this automatically when destroyed?

+}
 
 ////////////////////////////////////////////////////////////////////////////////
 // cc::mojom::MojoCompositorFrameSink overrides:
 
 void CompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
   support_.SetNeedsBeginFrame(needs_begin_frame);
 }
 
 void CompositorFrameSink::SubmitCompositorFrame(
     const cc::LocalFrameId& local_frame_id,
@@ skipping 28 matching lines @@
 void CompositorFrameSink::ReclaimResources(
     const cc::ReturnedResourceArray& resources) {
   client_->ReclaimResources(resources);
 }
 
 void CompositorFrameSink::WillDrawSurface() {
   client_->WillDrawSurface();
 }
 
 }  // namespace exo