Adding GCM support to the copresence component.

components/copresence/handlers/gcm_handler.h

Index: components/copresence/handlers/gcm_handler.h
diff --git a/components/copresence/handlers/gcm_handler.h b/components/copresence/handlers/gcm_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..35924a2e12d55ccbfb5fd79a6211ab5b309352d0
--- /dev/null
+++ b/components/copresence/handlers/gcm_handler.h
@@ -0,0 +1,81 @@
+// 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_COPRESENCE_HANDLERS_GCM_HANDLER_H_
+#define COMPONENTS_COPRESENCE_HANDLERS_GCM_HANDLER_H_
+
+#include <string>
+#include <vector>
+
+#include "base/callback_forward.h"
+#include "base/cancelable_callback.h"
+#include "components/gcm_driver/gcm_app_handler.h"
+#include "components/gcm_driver/gcm_client.h"
+
+namespace gcm {
+class GCMDriver;
+}
+
+namespace copresence {
+
+class DirectiveHandler;
+
+// This class handles GCM messages from the Copresence server.
+class GCMHandler final : public gcm::GCMAppHandler {
+ public:
+  // Callback to report that registration has completed.
+  // Returns an empty ID if registration failed.
+  using RegistrationCallback = base::Callback<void(const std::string&)>;
+
+  static const char kCopresenceAppId[];
+  static const char kCopresenceSenderId[];
+  static const char kGcmMessageKey[];
+
+  // |gcm_driver| is required, but may disappear if we get a ShutdownHandler()
+  // call first. |directive_handler| must outlive us. The caller owns both.
+  GCMHandler(gcm::GCMDriver* gcm_driver,
+             DirectiveHandler* directive_handler);
+  ~GCMHandler() override;
+
+  // Request the GCM ID. It may be returned now or later, via the callback.
+  void GetGcmId(const RegistrationCallback& callback);
+
+  // GCMAppHandler implementation
+

[rkc, 2014/11/07 20:24:15]

Remove all the newlines between these functions. Makes it easier to identify the
overrides. Also, change the comment to GCMAppHandler overrides: to keep it
consistent with the use everywhere else (unless you prefer to change everywhere
else to be "XXXX implementation.").

[Charlie, 2014/11/07 20:50:12]

Done.

+  void ShutdownHandler() override;
+
+  void OnMessage(const std::string& app_id,
+                 const gcm::GCMClient::IncomingMessage& message) override;
+  void OnMessagesDeleted(const std::string& app_id) override;
+
+  void OnSendError(
+      const std::string& app_id,
+      const gcm::GCMClient::SendErrorDetails& send_error_details) override;
+  void OnSendAcknowledged(const std::string& app_id,
+                          const std::string& message_id) override;
+
+  bool CanHandle(const std::string& app_id) const override;
+
+ private:
+  // GCM Registration has finished. Notify clients as appropriate.
+  void RegistrationComplete(const std::string& registration_id,
+                            gcm::GCMClient::Result result);
+
+  gcm::GCMDriver* driver_;
+  DirectiveHandler* const directive_handler_;
+
+  // If we are destructed before GCM registration completes,
+  // we cancel this callback.
+  base::CancelableCallback<void(const std::string&,
+                                gcm::GCMClient::Result)> registration_callback_;
+
+  std::string gcm_id_;
+  std::vector<RegistrationCallback> pending_id_requests_;
+
+  DISALLOW_COPY_AND_ASSIGN(GCMHandler);
+};
+
+}  // namespace copresence
+
+#endif  // COMPONENTS_COPRESENCE_HANDLERS_GCM_HANDLER_H_