Shift URLRequestContextStorage over to taking scoped_ptrs.

components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.cc

Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.cc
index 4a37e8ee371e9c5dd8652839d0b48d0dc66f22eb..60f41c6305dbf0d41fad2db5529738b0aecdb8eb 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_test_utils.cc
@@ -527,16 +527,17 @@ void DataReductionProxyTestContext::AttachToURLRequestContext(
 
   // |request_context_storage| takes ownership of the network delegate.
   request_context_storage->set_network_delegate(
-      io_data()->CreateNetworkDelegate(
-          scoped_ptr<net::NetworkDelegate>(new net::TestNetworkDelegate()),
-          true).release());
+      io_data()

[Deprecated (see juliatuttle), 2015/09/08 15:39:42]

Is this what git cl format does? Separating the io_data() and the
->CreateNetworkDelegate seems unnecessary.

[Randy Smith (Not in Mondays), 2015/09/08 16:33:17]

Yeah, that's git cl format (changed back, re-ran, it corrected me again).  I'll
try to fix it and upload and see if any presubmit stuff objects.  ETA: Nope,
this directory requires using git cl format; I'm afraid that I'm just going to
take whatever it gives me.

+          ->CreateNetworkDelegate(
+              scoped_ptr<net::NetworkDelegate>(new net::TestNetworkDelegate()),

[pauljensen, 2015/08/28 13:38:02]

I see lots of places in this CL where you've added scoped_ptr<>() instead of
make_scoped_ptr(), I assume because an implicit upcast is needed.
I wonder if this could be changed to make_scoped_ptr(...).Pass()?
We used to use .PassAs<>() but that was eliminated; now just Pass() is required.
This would make things more consistent (i.e. using make_scoped_ptr in all
places).
Some directories forbid scoped_ptr<>():
https://code.google.com/p/chromium/codesearch#chromium/src/ui/PRESUBMIT.py&l=34

[Randy Smith (Not in Mondays), 2015/09/02 23:42:19]

So "Done", with the exception of places where I'm returning a null scoped_ptr of
a particular type; I'm not sure how to do that with make_scoped_ptr.  An example
is LayoutTestURLRequestContextGetter::GetProxyConfigService().  Any ideas of
what the right idiom is in that case?

+              true)
+          .Pass());
 
-  // |request_context_storage| takes ownership of the job factory.
   request_context_storage->set_job_factory(
-      new net::URLRequestInterceptingJobFactory(
+      make_scoped_ptr(new net::URLRequestInterceptingJobFactory(
           scoped_ptr<net::URLRequestJobFactory>(
               new net::URLRequestJobFactoryImpl()),
-          io_data()->CreateInterceptor().Pass()));
+          io_data()->CreateInterceptor().Pass())));
 }
 
 void DataReductionProxyTestContext::