Using scoped_ptr for URLRequestJobFactoryImpl::SetProtocolHandler

content/browser/service_worker/service_worker_url_request_job_unittest.cc

Index: content/browser/service_worker/service_worker_url_request_job_unittest.cc
diff --git a/content/browser/service_worker/service_worker_url_request_job_unittest.cc b/content/browser/service_worker/service_worker_url_request_job_unittest.cc
index 149c524e7435ff456ec21f97d7853a80878eb02f..3a4e7a70732bfaf7b3ca445184a49c9969fb9938 100644
--- a/content/browser/service_worker/service_worker_url_request_job_unittest.cc
+++ b/content/browser/service_worker/service_worker_url_request_job_unittest.cc
@@ -183,11 +183,12 @@ class ServiceWorkerURLRequestJobTest : public testing::Test {
     url_request_job_factory_.reset(new net::URLRequestJobFactoryImpl);
     url_request_job_factory_->SetProtocolHandler(
         "http",
-        new MockHttpProtocolHandler(provider_host->AsWeakPtr(),
-                                    browser_context_->GetResourceContext(),
-                                    blob_storage_context->AsWeakPtr()));
+        make_scoped_ptr(new MockHttpProtocolHandler(
+            provider_host->AsWeakPtr(), browser_context_->GetResourceContext(),
+            blob_storage_context->AsWeakPtr())));
     url_request_job_factory_->SetProtocolHandler(
-        "blob", CreateMockBlobProtocolHandler(blob_storage_context));
+        "blob",
+        make_scoped_ptr(CreateMockBlobProtocolHandler(blob_storage_context)));

[davidben, 2015/08/18 20:40:37]

CreateMockBlobProtocolHandler could just return a scoped_ptr.

[svaldez, 2015/08/18 22:12:44]

Done.

     url_request_context_.set_job_factory(url_request_job_factory_.get());
 
     helper_->context()->AddProviderHost(provider_host.Pass());