Privatize StrongBinding lifetime management

chrome/gpu/gpu_arc_video_service.h

 // 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.
 
 #ifndef CHROME_GPU_GPU_ARC_VIDEO_SERVICE_H_
 #define CHROME_GPU_GPU_ARC_VIDEO_SERVICE_H_
 
 #include <memory>
 
 #include "base/files/scoped_file.h"
 #include "base/macros.h"
 #include "chrome/gpu/arc_video_accelerator.h"
 #include "components/arc/common/video_accelerator.mojom.h"
 #include "gpu/command_buffer/service/gpu_preferences.h"
-#include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace chromeos {
 namespace arc {
 
 // GpuArcVideoService manages life-cycle and IPC message translation for
 // ArcVideoAccelerator.
 //
 // For each creation request from GpuArcVideoServiceHost, GpuArcVideoService
 // will create a new IPC channel.
 class GpuArcVideoService : public ::arc::mojom::VideoAcceleratorService,
                            public ArcVideoAccelerator::Client {
  public:
-  GpuArcVideoService(
-      ::arc::mojom::VideoAcceleratorServiceRequest request,
-      const gpu::GpuPreferences& gpu_preferences);
   explicit GpuArcVideoService(const gpu::GpuPreferences& gpu_preferences);
   ~GpuArcVideoService() override;
 
   // Connects to VideoAcceleratorServiceClient.
   // |request| specified the message pipe of client to use.
-  void Connect(::arc::mojom::VideoAcceleratorServiceClientRequest request);
+  static void DeprecatedConnect(
+      std::unique_ptr<GpuArcVideoService> service,
+      ::arc::mojom::VideoAcceleratorServiceClientRequest client_request);
 
  private:
   // ArcVideoAccelerator::Client implementation.
   void OnError(ArcVideoAccelerator::Result error) override;
   void OnBufferDone(PortType port,
                     uint32_t index,
                     const BufferMetadata& metadata) override;
   void OnFlushDone() override;
   void OnResetDone() override;
   void OnOutputFormatChanged(const VideoFormat& format) override;
@@ skipping 27 matching lines @@
   void Reset() override;
 
   base::ScopedFD UnwrapFdFromMojoHandle(mojo::ScopedHandle handle);
 
   base::ThreadChecker thread_checker_;
 
   gpu::GpuPreferences gpu_preferences_;
   std::unique_ptr<ArcVideoAccelerator> accelerator_;
   ::arc::mojom::VideoAcceleratorServiceClientPtr client_;
 
-  // Binding of arc::mojom::VideoAcceleratorService. It also takes ownership of
-  // |this|.
-  mojo::StrongBinding<::arc::mojom::VideoAcceleratorService> binding_;
-
   DISALLOW_COPY_AND_ASSIGN(GpuArcVideoService);
 };
 
 }  // namespace arc
 }  // namespace chromeos
 
 #endif  // CHROME_GPU_GPU_ARC_VIDEO_SERVICE_H_