[GCM] Start GCMChannelStatusSyncer when GCM is disabled

components/gcm_driver/gcm_channel_status_request.h

Index: components/gcm_driver/gcm_channel_status_request.h
diff --git a/components/gcm_driver/gcm_channel_status_request.h b/components/gcm_driver/gcm_channel_status_request.h
index c2f41932e865cd5a16e4837a2d11aa426af2ef43..64d342f958b57f370860e48b89112f9c4f903fbe 100644
--- a/components/gcm_driver/gcm_channel_status_request.h
+++ b/components/gcm_driver/gcm_channel_status_request.h
@@ -26,7 +26,14 @@ namespace gcm {
 class GCMChannelStatusRequest : public net::URLFetcherDelegate {
  public:
   // Callback completing the channel status request.
-  typedef base::Callback<void(bool enabled, int poll_interval_seconds)>
+  // |update_received|: use the existing values if it is false which means no
+  //                    update is received.
+  // |enabled|: indicates if GCM should be enabled or not.

[fgorski, 2014/10/14 17:40:37]

nit: or disabled.

Why? One could read that statement as: Should I enable GCM? yes/no, as opposed
to: Is GCM allowed (enabled) to run? yes/no

[jianli, 2014/10/14 23:21:47]

Updated the comment.

+  // |poll_interval_seconds|: the interval in seconds to start next polling

[fgorski, 2014/10/14 17:40:37]

nit: next poll request

[jianli, 2014/10/14 23:21:47]

Done.

+  //                          request.
+  typedef base::Callback<void(bool update_received,
+                              bool enabled,
+                              int poll_interval_seconds)>
       GCMChannelStatusRequestCallback;
 
   GCMChannelStatusRequest(