Privatize StrongBinding lifetime management

extensions/browser/api/mime_handler_private/mime_handler_private.cc

 // Copyright 2015 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 "extensions/browser/api/mime_handler_private/mime_handler_private.h"
 
 #include <utility>
 
 #include "base/strings/string_util.h"
 #include "content/public/browser/stream_handle.h"
 #include "content/public/browser/stream_info.h"
 #include "content/public/common/content_constants.h"
 #include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.h"
 #include "extensions/common/constants.h"
 #include "mojo/public/cpp/bindings/map.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 #include "net/http/http_response_headers.h"
 
 namespace extensions {
 namespace {
 
 mojo::Map<mojo::String, mojo::String> CreateResponseHeadersMap(
     const net::HttpResponseHeaders* headers) {
   std::map<std::string, std::string> result;
   if (!headers)
     return mojo::Map<mojo::String, mojo::String>::From(result);
@@ skipping 14 matching lines @@
     auto& current_value = result[header_name];
     if (!current_value.empty())
       current_value += ", ";
     current_value += header_value;
   }
   return mojo::Map<mojo::String, mojo::String>::From(result);
 }
 
 }  // namespace
 
+MimeHandlerServiceImpl::MimeHandlerServiceImpl(
+    base::WeakPtr<StreamContainer> stream_container)
+    : stream_(stream_container), weak_factory_(this) {}
+
+MimeHandlerServiceImpl::~MimeHandlerServiceImpl() {}
+
 // static
 void MimeHandlerServiceImpl::Create(
     base::WeakPtr<StreamContainer> stream_container,
-    mojo::InterfaceRequest<mime_handler::MimeHandlerService> request) {
-  new MimeHandlerServiceImpl(stream_container, std::move(request));
-}
-
-MimeHandlerServiceImpl::MimeHandlerServiceImpl(
-    base::WeakPtr<StreamContainer> stream_container,
-    mojo::InterfaceRequest<mime_handler::MimeHandlerService> request)
-    : stream_(stream_container),
-      binding_(this, std::move(request)),
-      weak_factory_(this) {}
-
-MimeHandlerServiceImpl::~MimeHandlerServiceImpl() {
+    mime_handler::MimeHandlerServiceRequest request) {
+  mojo::MakeStrongBinding(
+      base::MakeUnique<MimeHandlerServiceImpl>(stream_container),
+      std::move(request));
 }
 
 void MimeHandlerServiceImpl::GetStreamInfo(
     const GetStreamInfoCallback& callback) {
   if (!stream_) {
     callback.Run(mime_handler::StreamInfoPtr());
     return;
   }
   callback.Run(mojo::ConvertTo<mime_handler::StreamInfoPtr>(*stream_));
 }
@@ skipping 40 matching lines @@
     result->original_url = info->original_url.spec();
   }
 
   result->stream_url = info->handle->GetURL().spec();
   result->response_headers =
       extensions::CreateResponseHeadersMap(info->response_headers.get());
   return result;
 }
 
 }  // namespace mojo