Privatize StrongBinding lifetime management

content/browser/media/capture/image_capture_impl.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/media/capture/image_capture_impl.h"
 
 #include <utility>
 
 #include "base/bind_helpers.h"
 #include "content/browser/browser_main_loop.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/browser/renderer_host/media/video_capture_manager.h"
 #include "content/common/media/media_stream_options.h"
 #include "content/public/browser/browser_thread.h"
 #include "media/base/bind_to_current_loop.h"
 #include "media/capture/video/video_capture_device.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace content {
 
 namespace {
 
 void RunGetCapabilitiesCallbackOnUIThread(
     const ImageCaptureImpl::GetCapabilitiesCallback& callback,
     media::mojom::PhotoCapabilitiesPtr capabilities) {
   BrowserThread::PostTask(
       BrowserThread::UI, FROM_HERE,
@@ skipping 82 matching lines @@
       media_stream_manager->VideoDeviceIdToSessionId(source_id);
 
   if (session_id == StreamDeviceInfo::kNoId)
     return;
   media_stream_manager->video_capture_manager()->TakePhoto(session_id,
                                                            std::move(callback));
 }
 
 }  // anonymous namespace
 
-// static
-void ImageCaptureImpl::Create(
-    mojo::InterfaceRequest<media::mojom::ImageCapture> request) {
-  // |binding_| will take ownership of ImageCaptureImpl.
-  new ImageCaptureImpl(std::move(request));
-}
+ImageCaptureImpl::ImageCaptureImpl() {}
 
 ImageCaptureImpl::~ImageCaptureImpl() {}
 
+// static
+void ImageCaptureImpl::Create(media::mojom::ImageCaptureRequest request) {
+  mojo::MakeStrongBinding(base::MakeUnique<ImageCaptureImpl>(),
+                          std::move(request));
+}
+
 void ImageCaptureImpl::GetCapabilities(
     const std::string& source_id,
     const GetCapabilitiesCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   media::ScopedResultCallback<GetCapabilitiesCallback> scoped_callback(
       base::Bind(&RunGetCapabilitiesCallbackOnUIThread, callback),
       media::BindToCurrentLoop(base::Bind(&RunFailedGetCapabilitiesCallback)));
 
   BrowserThread::PostTask(
@@ skipping 27 matching lines @@
       base::Bind(&RunTakePhotoCallbackOnUIThread, callback),
       media::BindToCurrentLoop(base::Bind(&RunFailedTakePhotoCallback)));
 
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&TakePhotoOnIOThread, source_id,
                  BrowserMainLoop::GetInstance()->media_stream_manager(),
                  base::Passed(&scoped_callback)));
 }
 
-ImageCaptureImpl::ImageCaptureImpl(
-    mojo::InterfaceRequest<media::mojom::ImageCapture> request)
-    : binding_(this, std::move(request)) {}
-
 }  // namespace content