Privatize StrongBinding lifetime management

services/shell/standalone/context.cc

 // Copyright 2013 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 "services/shell/standalone/context.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <utility>
@@ skipping 45 matching lines @@
   Setup() { mojo::edk::Init(); }
 
   ~Setup() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(Setup);
 };
 
 class TracingInterfaceProvider : public mojom::InterfaceProvider {
  public:
-  TracingInterfaceProvider(Tracer* tracer,
-                           mojom::InterfaceProviderRequest request)
-      : tracer_(tracer), binding_(this, std::move(request)) {}
+  explicit TracingInterfaceProvider(Tracer* tracer) : tracer_(tracer) {}
   ~TracingInterfaceProvider() override {}
 
   // mojom::InterfaceProvider:
   void GetInterface(const std::string& interface_name,
                     mojo::ScopedMessagePipeHandle client_handle) override {
     if (tracer_ && interface_name == tracing::mojom::Provider::Name_) {
       tracer_->ConnectToProvider(
           mojo::MakeRequest<tracing::mojom::Provider>(
               std::move(client_handle)));
     }
   }
 
  private:
   Tracer* tracer_;
-  mojo::StrongBinding<mojom::InterfaceProvider> binding_;
 
   DISALLOW_COPY_AND_ASSIGN(TracingInterfaceProvider);
 };
 
 const size_t kMaxBlockingPoolThreads = 3;
 
 std::unique_ptr<base::Thread> CreateIOThread(const char* name) {
   std::unique_ptr<base::Thread> thread(new base::Thread(name));
   base::Thread::Options options;
   options.message_loop_type = base::MessageLoop::TYPE_IO;
@@ skipping 73 matching lines @@
   std::unique_ptr<catalog::Store> store;
   if (init_params)
     store = std::move(init_params->catalog_store);
   catalog_.reset(
       new catalog::Catalog(blocking_pool_.get(), std::move(store), nullptr));
   service_manager_.reset(new ServiceManager(std::move(runner_factory),
                                             catalog_->TakeService()));
 
   mojom::InterfaceProviderPtr tracing_remote_interfaces;
   mojom::InterfaceProviderPtr tracing_local_interfaces;
-  new TracingInterfaceProvider(&tracer_, GetProxy(&tracing_local_interfaces));
+  mojo::MakeStrongBinding(base::MakeUnique<TracingInterfaceProvider>(&tracer_),
+                          mojo::GetProxy(&tracing_local_interfaces));
 
   std::unique_ptr<ConnectParams> params(new ConnectParams);
   params->set_source(CreateServiceManagerIdentity());
   params->set_target(Identity("mojo:tracing", mojom::kRootUserID));
   params->set_remote_interfaces(mojo::GetProxy(&tracing_remote_interfaces));
   service_manager_->Connect(std::move(params));
 
   if (command_line.HasSwitch(tracing::kTraceStartup)) {
     tracing::mojom::CollectorPtr coordinator;
     auto coordinator_request = GetProxy(&coordinator);
@@ skipping 67 matching lines @@
   mojom::InterfaceProviderPtr local_interfaces;
 
   std::unique_ptr<ConnectParams> params(new ConnectParams);
   params->set_source(CreateServiceManagerIdentity());
   params->set_target(Identity(name, mojom::kRootUserID));
   params->set_remote_interfaces(mojo::GetProxy(&remote_interfaces));
   service_manager_->Connect(std::move(params));
 }
 
 }  // namespace shell