Remove reference counting from HttpNetworkSession.

android_webview/browser/net/aw_url_request_context_getter.h

Index: android_webview/browser/net/aw_url_request_context_getter.h
diff --git a/android_webview/browser/net/aw_url_request_context_getter.h b/android_webview/browser/net/aw_url_request_context_getter.h
index a7713fe83c0287bd001a807ab5599005419ea8dc..868a67019232113806594958bfdb1739b99407c4 100644
--- a/android_webview/browser/net/aw_url_request_context_getter.h
+++ b/android_webview/browser/net/aw_url_request_context_getter.h
@@ -65,13 +65,15 @@ class AwURLRequestContextGetter : public net::URLRequestContextGetter {
   void InitializeURLRequestContext();
 
   const base::FilePath cache_path_;
-  scoped_refptr<net::CookieStore> cookie_store_;
+
   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::URLRequestContext> url_request_context_;
   scoped_ptr<net::ProxyConfigService> proxy_config_service_;
-  scoped_ptr<net::URLRequestJobFactory> job_factory_;
+  scoped_ptr<net::HttpNetworkSession> http_network_session_;
   scoped_ptr<net::HttpTransactionFactory> main_http_factory_;
+  scoped_refptr<net::CookieStore> cookie_store_;
+  scoped_ptr<net::URLRequestJobFactory> job_factory_;
   scoped_ptr<net::HttpUserAgentSettings> http_user_agent_settings_;
+  scoped_ptr<net::URLRequestContext> url_request_context_;
 
   // ProtocolHandlers and interceptors are stored here between
   // SetHandlersAndInterceptors() and the first GetURLRequestContext() call.