Privatize StrongBinding lifetime management

chrome/gpu/chrome_content_gpu_client.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 "chrome/gpu/chrome_content_gpu_client.h"
 
 #include "base/command_line.h"
+#include "base/memory/ptr_util.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 #include "services/shell/public/cpp/connector.h"
 #include "services/shell/public/cpp/interface_registry.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/gpu/gpu_arc_video_service.h"
 #endif
 
 #if defined(OS_CHROMEOS)
 namespace {
 
 void DeprecatedCreateGpuArcVideoService(
     const gpu::GpuPreferences& gpu_preferences,
-    ::arc::mojom::VideoAcceleratorServiceClientRequest request) {
-  // GpuArcVideoService is strongly bound to the Mojo message pipe it
-  // is connected to. When that message pipe is closed, either explicitly on the
-  // other end (in the browser process), or by a connection error, this object
-  // will be destroyed.
-  auto* service = new chromeos::arc::GpuArcVideoService(gpu_preferences);
-  service->Connect(std::move(request));
+    ::arc::mojom::VideoAcceleratorServiceClientRequest client_request) {
+  chromeos::arc::GpuArcVideoService::DeprecatedConnect(
+      base::MakeUnique<chromeos::arc::GpuArcVideoService>(gpu_preferences),
+      std::move(client_request));
 }
 
 void CreateGpuArcVideoService(
     const gpu::GpuPreferences& gpu_preferences,
     ::arc::mojom::VideoAcceleratorServiceRequest request) {
-  // GpuArcVideoService is strongly bound to the Mojo message pipe it
-  // is connected to. When that message pipe is closed, either explicitly on the
-  // other end (in the browser process), or by a connection error, this object
-  // will be destroyed.
-  new chromeos::arc::GpuArcVideoService(std::move(request), gpu_preferences);
+  mojo::MakeStrongBinding(
+      base::MakeUnique<chromeos::arc::GpuArcVideoService>(gpu_preferences),
+      std::move(request));
 }
 
 }  // namespace
 #endif
 
 ChromeContentGpuClient::ChromeContentGpuClient() {}
 
 ChromeContentGpuClient::~ChromeContentGpuClient() {}
 
 void ChromeContentGpuClient::Initialize(
@@ skipping 13 matching lines @@
   registry->AddInterface(
       base::Bind(&CreateGpuArcVideoService, gpu_preferences));
   registry->AddInterface(
       base::Bind(&DeprecatedCreateGpuArcVideoService, gpu_preferences));
 #endif
 }
 
 void ChromeContentGpuClient::ConsumeInterfacesFromBrowser(
     shell::InterfaceProvider* provider) {
 }