DirectCompositorFrameSink Uses CompositorFrameSinkSupport

components/exo/compositor_frame_sink.cc

Index: components/exo/compositor_frame_sink.cc
diff --git a/components/exo/compositor_frame_sink.cc b/components/exo/compositor_frame_sink.cc
index c49758a40667eb45d6afb09dccdbd76803fbb5ef..71d9612cb01eb3c8004c8c5ef84e09b1940f201a 100644
--- a/components/exo/compositor_frame_sink.cc
+++ b/components/exo/compositor_frame_sink.cc
@@ -18,10 +18,12 @@ namespace exo {
 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: