[GCM] Check-in with signed in accounts associates device to user

components/gcm_driver/gcm_driver_desktop.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 COMPONENTS_GCM_DRIVER_GCM_DRIVER_DESKTOP_H_
 #define COMPONENTS_GCM_DRIVER_GCM_DRIVER_DESKTOP_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 38 matching lines @@
   virtual ~GCMDriverDesktop();
 
   // GCMDriver overrides:
   virtual void Shutdown() OVERRIDE;
   virtual void OnSignedIn() OVERRIDE;
   virtual void Purge() OVERRIDE;
   virtual void AddAppHandler(const std::string& app_id,
                              GCMAppHandler* handler) OVERRIDE;
   virtual void RemoveAppHandler(const std::string& app_id) OVERRIDE;
 
   // GCMDriver implementation:

[Nicolas Zea, 2014/07/10 20:57:32]

should these two sections be combined (this and the one above)?

[fgorski, 2014/07/11 22:51:21]

Done.

   virtual void Enable() OVERRIDE;
   virtual void Disable() OVERRIDE;
   virtual GCMClient* GetGCMClientForTesting() const OVERRIDE;
   virtual bool IsStarted() const OVERRIDE;
   virtual bool IsConnected() const OVERRIDE;
   virtual void GetGCMStatistics(const GetGCMStatisticsCallback& callback,
                                 bool clear_logs) OVERRIDE;
   virtual void SetGCMRecording(const GetGCMStatisticsCallback& callback,
                                bool recording) OVERRIDE;
 
+  // GCMDriverDesktop specific implementation.
+  // Sets a list of accounts with OAuth2 tokens for the next checkin.
+  // |account_tokens| maps email addresses to OAuth2 access tokens.
+  // |account_removed| indicates that an account has been removed since the
+  //     last time the callback was called, which triggers an immediate checkin,
+  //     to ensure that association between device and account is removed.
+  void SetAccountsForCheckin(
+      const std::map<std::string, std::string>& account_tokens,
+      bool account_removed);

[Nicolas Zea, 2014/07/10 20:57:32]

nit: style guide prefers using an enum to a bool in a public method like this
(e.g. enum AccountStatus {ACCOUNT_REMOVED, NO_ACCOUNTS_REMOVED})

[fgorski, 2014/07/11 22:51:21]

Done.

+
  protected:
   // GCMDriver implementation:
   virtual GCMClient::Result EnsureStarted() OVERRIDE;
   virtual void RegisterImpl(
       const std::string& app_id,
       const std::vector<std::string>& sender_ids) OVERRIDE;
   virtual void UnregisterImpl(const std::string& app_id) OVERRIDE;
   virtual void SendImpl(const std::string& app_id,
                         const std::string& receiver_id,
                         const GCMClient::OutgoingMessage& message) OVERRIDE;
@@ skipping 56 matching lines @@
 
   // Used to pass a weak pointer to the IO worker.
   base::WeakPtrFactory<GCMDriverDesktop> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GCMDriverDesktop);
 };
 
 }  // namespace gcm
 
 #endif  // COMPONENTS_GCM_DRIVER_GCM_DRIVER_DESKTOP_H_