Support getting and deleting token for Instance ID.

google_apis/gcm/engine/registration_request.h

 // Copyright 2014 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 GOOGLE_APIS_GCM_ENGINE_REGISTRATION_REQUEST_H_
 #define GOOGLE_APIS_GCM_ENGINE_REGISTRATION_REQUEST_H_
 
 #include <map>
 #include <vector>
 
@@ skipping 41 matching lines @@
     // immediately above this line. Make sure to update the corresponding
     // histogram enum accordingly.
     STATUS_COUNT
   };
 
   // Callback completing the registration request.
   typedef base::Callback<void(Status status,
                               const std::string& registration_id)>
       RegistrationCallback;
 
-  // Details of the of the Registration Request. Only user's android ID and
-  // its serial number are optional and can be set to 0. All other parameters
-  // have to be specified to successfully complete the call.
-  struct GCM_EXPORT RequestInfo {
-    RequestInfo(uint64 android_id,
-                uint64 security_token,
-                const std::string& app_id,
-                const std::vector<std::string>& sender_ids);
-    ~RequestInfo();
+  // Encapsulates the common info about a registration/token request.
+  class GCM_EXPORT RequestInfo {
+   public:
+    RequestInfo();
+    virtual ~RequestInfo();
 
+    virtual void BuildRequestHeaders(std::string* extra_headers);

[Nicolas Zea, 2015/05/15 20:37:01]

Style guide prefers either doing pure inheritance (e.g. interfaces) or using
composition instead.

How about having the RequestInfo contain a the android id, security token,
chrome version, and app id, and then also a RequestId member. The RequestId
class would be a purely virtual class with two implementations like you have
below (and also any id implementation specific helper methods).

In some sense though, I think it might make sense to keep a RegistrationId class
around that can encapsulate ID information in general. See my comment about how
we're persisting it into the gcm store for example.

[jianli, 2015/05/21 00:41:15]

With more thought, I decided to make RequestInfo still defined as struct without
introducing any virtual methods. All Build*** methods have been moved into
RegistrationRequest. We now have GCMRegistrationRequest and
InstanceIDGetTokenRequest that derive from RegistrationRequest, which will
implement the different request building and response parsing logic.

+    virtual void BuildRequestBody(std::string* body);
+    virtual std::string GetSenders() const = 0;
+
+    std::string app_id() const { return app_id_; }
+
+    void set_chrome_version(const std::string& chrome_version) {
+        chrome_version_ = chrome_version;
+    }
+    void set_android_id(uint64 android_id) {
+        android_id_ = android_id;
+    }
+    void set_security_token(uint64 security_token) {
+        security_token_ = security_token;
+    }
+    void set_app_id(const std::string& app_id) {
+        app_id_ = app_id;
+    }
+
+   protected:
+    // Version to pass.
+    std::string chrome_version_;
     // Android ID of the device.
-    uint64 android_id;
+    uint64 android_id_;
     // Security token of the device.
-    uint64 security_token;
+    uint64 security_token_;
     // Application ID.
-    std::string app_id;
-    // Certificate of the application.
-    std::string cert;
+    std::string app_id_;
+
+   private:
+    DISALLOW_COPY_AND_ASSIGN(RequestInfo);
+  };
+
+  // GCM registration request.
+  class GCM_EXPORT GCMRequestInfo : public RequestInfo {
+   public:
+    GCMRequestInfo();
+    ~GCMRequestInfo() override;
+
+    // RequestInfo overrides:
+    void BuildRequestBody(std::string* body) override;
+    std::string GetSenders() const override;
+
+    void set_sender_ids(const std::vector<std::string>& sender_ids) {
+        sender_ids_ = sender_ids;
+    }
+
+   private:
     // List of IDs of senders. Allowed up to 100.
-    std::vector<std::string> sender_ids;
+    std::vector<std::string> sender_ids_;
+
+    DISALLOW_COPY_AND_ASSIGN(GCMRequestInfo);
+  };
+
+  // Instance ID token request.
+  class GCM_EXPORT InstanceIDRequestInfo : public RequestInfo {
+   public:
+    InstanceIDRequestInfo();
+    ~InstanceIDRequestInfo() override;
+
+    // RequestInfo overrides:
+    void BuildRequestBody(std::string* body) override;
+    std::string GetSenders() const override;
+
+    void set_instance_id(const std::string& instance_id) {
+        instance_id_ = instance_id;
+    }
+    void set_authorized_entity(const std::string& authorized_entity) {
+        authorized_entity_ = authorized_entity;
+    }
+    void set_scope(const std::string& scope) {
+        scope_ = scope;
+    }
+    void set_options(const std::map<std::string, std::string>& options) {
+        options_ = options;
+    }
+
+   private:
+    std::string instance_id_;
+    std::string authorized_entity_;
+    std::string scope_;
+    std::map<std::string, std::string> options_;
+
+    DISALLOW_COPY_AND_ASSIGN(InstanceIDRequestInfo);
   };
 
   RegistrationRequest(
       const GURL& registration_url,
-      const RequestInfo& request_info,
+      scoped_ptr<RequestInfo> request_info,
       const net::BackoffEntry::Policy& backoff_policy,
       const RegistrationCallback& callback,
       int max_retry_count,
       scoped_refptr<net::URLRequestContextGetter> request_context_getter,
       GCMStatsRecorder* recorder);
   ~RegistrationRequest() override;
 
   void Start();
 
   // URLFetcherDelegate implementation.
   void OnURLFetchComplete(const net::URLFetcher* source) override;
 
  private:
   // Schedules a retry attempt, informs the backoff of a previous request's
   // failure, when |update_backoff| is true.
   void RetryWithBackoff(bool update_backoff);
 
   // Parse the response returned by the URL fetcher into token, and returns the
   // status.
   Status ParseResponse(const net::URLFetcher* source, std::string* token);
 
   RegistrationCallback callback_;
-  RequestInfo request_info_;
+  scoped_ptr<RequestInfo> request_info_;
   GURL registration_url_;
 
   net::BackoffEntry backoff_entry_;
   scoped_refptr<net::URLRequestContextGetter> request_context_getter_;
   scoped_ptr<net::URLFetcher> url_fetcher_;
   int retries_left_;
   base::TimeTicks request_start_time_;
 
   // Recorder that records GCM activities for debugging purpose. Not owned.
   GCMStatsRecorder* recorder_;
 
   base::WeakPtrFactory<RegistrationRequest> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RegistrationRequest);
 };
 
 }  // namespace gcm
 
 #endif  // GOOGLE_APIS_GCM_ENGINE_REGISTRATION_REQUEST_H_