exo::CompositorFrameSink uses CompositorFrameSinkHolder* instead of mojo interface pointer

components/exo/compositor_frame_sink.h

Index: components/exo/compositor_frame_sink.h
diff --git a/components/exo/compositor_frame_sink.h b/components/exo/compositor_frame_sink.h
index 41449cc0f0445b4df84067a44da35a30e132197e..86488d3b901220abaaf2dd613053adb9505edf1f 100644
--- a/components/exo/compositor_frame_sink.h
+++ b/components/exo/compositor_frame_sink.h
@@ -14,12 +14,14 @@
 
 namespace exo {
 
+class CompositorFrameSinkHolder;
+
 class CompositorFrameSink : public cc::CompositorFrameSinkSupportClient,
                             public cc::mojom::MojoCompositorFrameSink {
  public:
   CompositorFrameSink(const cc::FrameSinkId& frame_sink_id,
                       cc::SurfaceManager* surface_manager,
-                      cc::mojom::MojoCompositorFrameSinkClientPtr client);
+                      CompositorFrameSinkHolder* client);
 
   ~CompositorFrameSink() override;
 
@@ -44,7 +46,7 @@ class CompositorFrameSink : public cc::CompositorFrameSinkSupportClient,
 
  private:
   cc::CompositorFrameSinkSupport support_;
-  cc::mojom::MojoCompositorFrameSinkClientPtr client_;
+  CompositorFrameSinkHolder* client_;
   cc::ReturnedResourceArray surface_returned_resources_;
   mojo::StrongBindingPtr<cc::mojom::MojoCompositorFrameSink> binding_;

[Fady Samuel, 2017/01/13 19:14:28]

Try restoring the existing code and making this binding_ the first member
variable. Make CompositorFrameSinkSupport the last member variable.

This would cause CompositorFrameSinkSupport to be destroyed first, and
mojo::StrongBindingPtr<cc::mojom::MojoCompositorFrameSink> to be destroyed last.
I'm curious what would happen.