Privatize StrongBinding lifetime management

chrome/browser/chromeos/arc/gpu_arc_video_service_host.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/browser/chromeos/arc/gpu_arc_video_service_host.h"
 
 #include <string>
 
 #include "base/location.h"
 #include "base/logging.h"
@@ skipping 24 matching lines @@
 
 void ConnectToVideoAcceleratorServiceOnIOThread(
     mojom::VideoAcceleratorServiceRequest request) {
   content::GetGpuRemoteInterfaces()->GetInterface(std::move(request));
 }
 
 }  // namespace
 
 class VideoAcceleratorFactoryService : public mojom::VideoAcceleratorFactory {
  public:
-  explicit VideoAcceleratorFactoryService(
-      mojom::VideoAcceleratorFactoryRequest request)
-      : binding_(this, std::move(request)) {}
+  VideoAcceleratorFactoryService() = default;
 
   void Create(mojom::VideoAcceleratorServiceRequest request) override {
     content::BrowserThread::PostTask(
         content::BrowserThread::IO, FROM_HERE,
         base::Bind(&ConnectToVideoAcceleratorServiceOnIOThread,
                    base::Passed(&request)));
   }
 
  private:
-  mojo::StrongBinding<VideoAcceleratorFactory> binding_;
-
   DISALLOW_COPY_AND_ASSIGN(VideoAcceleratorFactoryService);
 };
 
 GpuArcVideoServiceHost::GpuArcVideoServiceHost(
     arc::ArcBridgeService* bridge_service)
     : ArcService(bridge_service), binding_(this) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   arc_bridge_service()->video()->AddObserver(this);
 }
 
@@ skipping 74 matching lines @@
   std::string token = mojo::edk::GenerateRandomToken();
   mojo::ScopedMessagePipeHandle server_pipe =
       mojo::edk::CreateParentMessagePipe(token, child_token);
   if (!server_pipe.is_valid()) {
     LOG(ERROR) << "Invalid pipe";
     callback.Run(mojo::ScopedHandle(), std::string());
     return;
   }
   callback.Run(std::move(child_handle), token);
 
-  // The lifetime is managed by the StrongBinding insides the
-  // VideoAcceleratorFactoryService.
-  new VideoAcceleratorFactoryService(
-      mojo::MakeRequest<mojom::VideoAcceleratorFactory>(
-          std::move(server_pipe)));
+  mojo::MakeStrongBinding(base::MakeUnique<VideoAcceleratorFactoryService>(),
+                          mojo::MakeRequest<mojom::VideoAcceleratorFactory>(
+                              std::move(server_pipe)));
 }
 
 }  // namespace arc