Manage OffscreenCanvasCompositorFrameSink lifetime explicitly

content/browser/renderer_host/offscreen_canvas_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 "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.h"
 
 #include "base/memory/ptr_util.h"
 #include "cc/surfaces/surface.h"
 #include "cc/surfaces/surface_manager.h"
+#include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace content {
 
-// static
-void OffscreenCanvasCompositorFrameSink::Create(
+OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink(
+    OffscreenCanvasCompositorFrameSinkProviderImpl* provider,
     const cc::FrameSinkId& frame_sink_id,
-    cc::SurfaceManager* surface_manager,
-    cc::mojom::MojoCompositorFrameSinkClientPtr client,
-    cc::mojom::MojoCompositorFrameSinkRequest request) {
-  std::unique_ptr<OffscreenCanvasCompositorFrameSink> impl =
-      base::MakeUnique<OffscreenCanvasCompositorFrameSink>(
-          frame_sink_id, surface_manager, std::move(client));
-  OffscreenCanvasCompositorFrameSink* compositor_frame_sink = impl.get();
-  compositor_frame_sink->binding_ =
-      mojo::MakeStrongBinding(std::move(impl), std::move(request));
+    cc::mojom::MojoCompositorFrameSinkRequest request,
+    cc::mojom::MojoCompositorFrameSinkClientPtr client)
+    : provider_(provider),
+      support_(this,
+               provider->GetSurfaceManager(),
+               frame_sink_id,
+               nullptr,
+               nullptr),
+      client_(std::move(client)),
+      binding_(this, std::move(request)) {
+  binding_.set_connection_error_handler(
+      base::Bind(&OffscreenCanvasCompositorFrameSink::OnClientConnectionLost,
+                 base::Unretained(this)));
 }
 
-OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink(
-    const cc::FrameSinkId& frame_sink_id,
-    cc::SurfaceManager* surface_manager,
-    cc::mojom::MojoCompositorFrameSinkClientPtr client)
-    : support_(this, surface_manager, frame_sink_id, nullptr, nullptr),
-      client_(std::move(client)) {}
-
 OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() {}
 
 void OffscreenCanvasCompositorFrameSink::SetNeedsBeginFrame(
     bool needs_begin_frame) {
   support_.SetNeedsBeginFrame(needs_begin_frame);
 }
 
 void OffscreenCanvasCompositorFrameSink::SubmitCompositorFrame(
     const cc::LocalFrameId& local_frame_id,
     cc::CompositorFrame frame) {
@@ skipping 42 matching lines @@
     const cc::ReturnedResourceArray& resources) {
   if (client_)
     client_->ReclaimResources(resources);
 }
 
 void OffscreenCanvasCompositorFrameSink::WillDrawSurface() {
   if (client_)
     client_->WillDrawSurface();
 }
 
+void OffscreenCanvasCompositorFrameSink::OnClientConnectionLost() {
+  provider_->OnCompositorFrameSinkClientConnectionLost(
+      support_.frame_sink_id());
+}
+
 }  // namespace content