Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1139)

Unified Diff: components/copresence/handlers/gcm_handler.h

Issue 713643002: Revert "Adding GCM support to the copresence component. The Copresence server can push directives (… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2214
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/copresence/copresence_manager_impl.cc ('k') | components/copresence/handlers/gcm_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/copresence/handlers/gcm_handler.h
diff --git a/components/copresence/handlers/gcm_handler.h b/components/copresence/handlers/gcm_handler.h
deleted file mode 100644
index f69d8a48090e181a46ce330763ee07f0ef862387..0000000000000000000000000000000000000000
--- a/components/copresence/handlers/gcm_handler.h
+++ /dev/null
@@ -1,78 +0,0 @@
-// 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.h"
-#include "base/memory/weak_ptr.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 base::SupportsWeakPtr<GCMHandler> {
- 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 overrides
- 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_;
-
- // TODO(ckehoe): Change this to a CancelableCallback.
- base::Callback<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_
« no previous file with comments | « components/copresence/copresence_manager_impl.cc ('k') | components/copresence/handlers/gcm_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698