Privatize StrongBinding lifetime management

extensions/renderer/api/serial/serial_api_unittest.cc

 // Copyright 2014 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 <stddef.h>
 #include <stdint.h>
 
 #include <utility>
 
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "device/serial/serial_device_enumerator.h"
 #include "device/serial/serial_service_impl.h"
 #include "device/serial/test_serial_io_handler.h"
 #include "extensions/browser/mojo/stash_backend.h"
 #include "extensions/common/mojo/keep_alive.mojom.h"
 #include "extensions/renderer/api_test_base.h"
 #include "grit/extensions_renderer_resources.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 
 // A test launcher for tests for the serial API defined in
 // extensions/test/data/serial_unittest.js. Each C++ test function sets up a
 // fake DeviceEnumerator or SerialIoHandler expecting or returning particular
 // values for that test.
 
 namespace extensions {
 
 namespace {
 
@@ skipping 398 matching lines @@
 
  private:
   scoped_refptr<device::SerialIoHandler> GetIoHandler() {
     if (!io_handler_.get())
       io_handler_ = new TestIoHandlerBase;
     return io_handler_;
   }
 
   void CreateSerialService(
       mojo::InterfaceRequest<device::serial::SerialService> request) {
-    new device::SerialServiceImpl(
-        new device::SerialConnectionFactory(
-            base::Bind(&SerialApiTest::GetIoHandler, base::Unretained(this)),
-            base::ThreadTaskRunnerHandle::Get()),
-        std::unique_ptr<device::SerialDeviceEnumerator>(
-            new FakeSerialDeviceEnumerator),
-        std::move(request));
+    mojo::MakeStrongBinding(base::MakeUnique<device::SerialServiceImpl>(
+                                new device::SerialConnectionFactory(
+                                    base::Bind(&SerialApiTest::GetIoHandler,
+                                               base::Unretained(this)),
+                                    base::ThreadTaskRunnerHandle::Get()),
+                                base::MakeUnique<FakeSerialDeviceEnumerator>()),
+                            std::move(request));
   }
 
   DISALLOW_COPY_AND_ASSIGN(SerialApiTest);
 };
 
 TEST_F(SerialApiTest, GetDevices) {
   RunTest("serial_unittest.js", "testGetDevices");
 }
 
 TEST_F(SerialApiTest, ConnectFail) {
@@ skipping 276 matching lines @@
   ASSERT_NO_FATAL_FAILURE(
       RunTest("serial_unittest.js", "testStashNoConnections"));
   io_handler_ = nullptr;
   std::unique_ptr<ModuleSystemTestEnvironment> new_env(CreateEnvironment());
   ApiTestEnvironment new_api_test_env(new_env.get());
   PrepareEnvironment(&new_api_test_env, stash_backend_.get());
   new_api_test_env.RunTest("serial_unittest.js", "testRestoreNoConnections");
 }
 
 }  // namespace extensions