Privatize StrongBinding lifetime management

services/navigation/navigation.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 "services/navigation/navigation.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "base/threading/thread_task_runner_handle.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 #include "services/navigation/view_impl.h"
 #include "services/shell/public/cpp/connector.h"
 
 namespace navigation {
 
 namespace {
 
 void CreateViewOnViewTaskRunner(
     std::unique_ptr<shell::Connector> connector,
     const std::string& client_user_id,
     mojom::ViewClientPtr client,
     mojom::ViewRequest request,
     std::unique_ptr<shell::ServiceContextRef> context_ref) {
-  // Owns itself.
-  new ViewImpl(std::move(connector), client_user_id, std::move(client),
-               std::move(request), std::move(context_ref));
+  mojo::MakeStrongBinding(
+      base::MakeUnique<ViewImpl>(std::move(connector), client_user_id,
+                                 std::move(client), std::move(context_ref)),
+      std::move(request));
 }
 
 }  // namespace
 
 Navigation::Navigation()
     : view_task_runner_(base::ThreadTaskRunnerHandle::Get()),
       ref_factory_(base::MessageLoop::QuitWhenIdleClosure()),
       weak_factory_(this) {
   bindings_.set_connection_error_handler(
       base::Bind(&Navigation::ViewFactoryLost, base::Unretained(this)));
@@ skipping 31 matching lines @@
 void Navigation::CreateViewFactory(mojom::ViewFactoryRequest request) {
   bindings_.AddBinding(this, std::move(request));
   refs_.insert(ref_factory_.CreateRef());
 }
 
 void Navigation::ViewFactoryLost() {
   refs_.erase(refs_.begin());
 }
 
 }  // navigation