Privatize StrongBinding lifetime management

content/browser/loader/url_loader_factory_impl.cc

Index: content/browser/loader/url_loader_factory_impl.cc
diff --git a/content/browser/loader/url_loader_factory_impl.cc b/content/browser/loader/url_loader_factory_impl.cc
index 2f02691c88816520504ff363a035a7c91b451add..d232f5cfe4452899ab60be5e508802a3f6aba46e 100644
--- a/content/browser/loader/url_loader_factory_impl.cc
+++ b/content/browser/loader/url_loader_factory_impl.cc
@@ -9,14 +9,13 @@
 #include "content/common/resource_request.h"
 #include "content/common/url_loader.mojom.h"
 #include "content/public/browser/browser_thread.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace content {
 
 URLLoaderFactoryImpl::URLLoaderFactoryImpl(
-    scoped_refptr<ResourceMessageFilter> resource_message_filter,
-    mojo::InterfaceRequest<mojom::URLLoaderFactory> request)
-    : resource_message_filter_(std::move(resource_message_filter)),
-      binding_(this, std::move(request)) {
+    scoped_refptr<ResourceMessageFilter> resource_message_filter)
+    : resource_message_filter_(std::move(resource_message_filter)) {
   DCHECK(resource_message_filter_);
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 }
@@ -43,8 +42,9 @@ void URLLoaderFactoryImpl::CreateLoaderAndStart(
 void URLLoaderFactoryImpl::Create(
     scoped_refptr<ResourceMessageFilter> filter,
     mojo::InterfaceRequest<mojom::URLLoaderFactory> request) {
-  // The created instance is held by the StrongBinding.
-  new URLLoaderFactoryImpl(std::move(filter), std::move(request));
+  mojo::MakeStrongBinding(
+      base::WrapUnique(new URLLoaderFactoryImpl(std::move(filter))),
+      std::move(request));
 }
 
 }  // namespace content