Remove reference counting from HttpNetworkSession.

components/gcm_driver/gcm_client.h

Index: components/gcm_driver/gcm_client.h
diff --git a/components/gcm_driver/gcm_client.h b/components/gcm_driver/gcm_client.h
index 0aa9a1a8f7520a73d885476f56a81e937ee6e3a0..5f7f293d7f144a6365732f287e61e2d536a71507 100644
--- a/components/gcm_driver/gcm_client.h
+++ b/components/gcm_driver/gcm_client.h
@@ -217,9 +217,10 @@ class GCMClient {
   // |chrome_build_info|: chrome info, i.e., version, channel and etc.
   // |store_path|: path to the GCM store.
   // |blocking_task_runner|: for running blocking file tasks.
-  // |url_request_context_getter|: for url requests.
+  // |url_request_context_getter|: for url requests. The GCMClient must be
+  //     deleted before the Getter's underlying URLRequestContext.

[mmenke, 2015/08/31 15:55:56]

This may seem like a scary new requirement, but we're actually already depending
on it.  The HttpNetworkSession may nominally have been refcounted, but it wasn't
safe to use after ProfileIOData was deleting, since HttpNetworkSession::Params
contains a bunch of raw pointers.

[Nicolas Zea, 2015/08/31 19:43:28]

Acknowledged.

   // |delegate|: the delegate whose methods will be called asynchronously in
-  //             response to events and messages.
+  //     response to events and messages.
   virtual void Initialize(
       const ChromeBuildInfo& chrome_build_info,
       const base::FilePath& store_path,