Remove reference counting from HttpNetworkSession.

android_webview/browser/net/aw_url_request_context_getter.h

 // Copyright (c) 2012 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.
 
 #ifndef ANDROID_WEBVIEW_BROWSER_NET_AW_URL_REQUEST_CONTEXT_GETTER_H_
 #define ANDROID_WEBVIEW_BROWSER_NET_AW_URL_REQUEST_CONTEXT_GETTER_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/files/file_path.h"
@@ skipping 47 matching lines @@
   // AwBrowserContext::CreateRequestContext() call this method.
   // This method is necessary because the passed in objects are created
   // on the UI thread while |job_factory_| must be created on the IO thread.
   void SetHandlersAndInterceptors(
       content::ProtocolHandlerMap* protocol_handlers,
       content::URLRequestInterceptorScopedVector request_interceptors);
 
   void InitializeURLRequestContext();
 
   const base::FilePath cache_path_;
+
+  scoped_ptr<net::NetLog> net_log_;

[pauljensen, 2015/09/02 14:32:29]

I like how you've reordered this here to better match required destruction order
and construction/dependence order.
nit: WDYT about changing net_log_ to simply be a net::NetLog instead of a
scoped_ptr<net::NetLog>?  Ditto for http_user_agent_settings_? Maybe not for
this CL.

[mmenke, 2015/09/02 16:29:33]

I think it makes sense, but is a bit beyond the scope of this CL.

+  scoped_ptr<net::ProxyConfigService> proxy_config_service_;
+  scoped_ptr<net::HttpNetworkSession> http_network_session_;
+  scoped_ptr<net::HttpTransactionFactory> main_http_factory_;
   scoped_refptr<net::CookieStore> cookie_store_;
-  scoped_ptr<net::NetLog> net_log_;
+  scoped_ptr<net::URLRequestJobFactory> job_factory_;
+  scoped_ptr<net::HttpUserAgentSettings> http_user_agent_settings_;
   scoped_ptr<net::URLRequestContext> url_request_context_;
-  scoped_ptr<net::ProxyConfigService> proxy_config_service_;
-  scoped_ptr<net::URLRequestJobFactory> job_factory_;
-  scoped_ptr<net::HttpTransactionFactory> main_http_factory_;
-  scoped_ptr<net::HttpUserAgentSettings> http_user_agent_settings_;
 
   // ProtocolHandlers and interceptors are stored here between
   // SetHandlersAndInterceptors() and the first GetURLRequestContext() call.
   content::ProtocolHandlerMap protocol_handlers_;
   content::URLRequestInterceptorScopedVector request_interceptors_;
 
   DISALLOW_COPY_AND_ASSIGN(AwURLRequestContextGetter);
 };
 
 }  // namespace android_webview
 
 #endif  // ANDROID_WEBVIEW_BROWSER_NET_AW_URL_REQUEST_CONTEXT_GETTER_H_