make ProxyService::CreateSystemProxyConfigService return scoped_ptrs

remoting/base/url_request_context_getter.cc

Index: remoting/base/url_request_context_getter.cc
diff --git a/remoting/base/url_request_context_getter.cc b/remoting/base/url_request_context_getter.cc
index b527481943dfdc860a8681bd2b56b6639fb35b10..7906f616fe54502633c6a957174976f57bbdfaaf 100644
--- a/remoting/base/url_request_context_getter.cc
+++ b/remoting/base/url_request_context_getter.cc
@@ -17,8 +17,8 @@ URLRequestContextGetter::URLRequestContextGetter(
     scoped_refptr<base::SingleThreadTaskRunner> file_task_runner)
     : network_task_runner_(network_task_runner),
       file_task_runner_(file_task_runner) {
-  proxy_config_service_.reset(net::ProxyService::CreateSystemProxyConfigService(
-      network_task_runner_, file_task_runner));
+  proxy_config_service_ = net::ProxyService::CreateSystemProxyConfigService(
+      network_task_runner_, file_task_runner);

[mmenke, 2015/09/21 21:34:51]

nit:  While you're here, remove the underscore at the end of
network_task_runner_?  (Weird that we use an argument for one, and a member
variable for the other).

[mmenke, 2015/09/21 21:34:51]

nit:  Can you just put this in an initializer list?

[Charlie Harrison, 2015/09/22 18:00:28]

Done.

 }
 
 net::URLRequestContext* URLRequestContextGetter::GetURLRequestContext() {