Push API: Fix unsubscribing from GCM on Android

components/gcm_driver/gcm_driver.cc

 // 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.
 
 #include "components/gcm_driver/gcm_driver.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 51 matching lines @@
         app_id,
         normalized_sender_ids,
         unregister_iter->second);
     return;
   }
 
   RegisterImpl(app_id, normalized_sender_ids);
 }
 
 void GCMDriver::Unregister(const std::string& app_id,
+                           const std::vector<std::string>& sender_ids,
                            const UnregisterCallback& callback) {
   DCHECK(!app_id.empty());
+#if defined(OS_ANDROID)
+  DCHECK(!sender_ids.empty());
+#endif
   DCHECK(!callback.is_null());
 
   GCMClient::Result result = EnsureStarted(GCMClient::IMMEDIATE_START);
   if (result != GCMClient::SUCCESS) {
     callback.Run(result);
     return;
   }
 
   // If previous un/register operation is still in progress, bail out.
   if (register_callbacks_.find(app_id) != register_callbacks_.end() ||
       unregister_callbacks_.find(app_id) != unregister_callbacks_.end()) {
     callback.Run(GCMClient::ASYNC_OPERATION_PENDING);
     return;
   }
 
+  // Normalize the sender IDs by making them sorted.
+  std::vector<std::string> normalized_sender_ids = sender_ids;
+  std::sort(normalized_sender_ids.begin(), normalized_sender_ids.end());
+
   unregister_callbacks_[app_id] = callback;
 
-  UnregisterImpl(app_id);
+  UnregisterImpl(app_id, normalized_sender_ids);
 }
 
 void GCMDriver::Send(const std::string& app_id,
                      const std::string& receiver_id,
                      const GCMClient::OutgoingMessage& message,
                      const SendCallback& callback) {
   DCHECK(!app_id.empty());
   DCHECK(!receiver_id.empty());
   DCHECK(!callback.is_null());
 
@@ skipping 114 matching lines @@
     GCMClient::Result result) {
   // Invoke the original unregister callback.
   unregister_callback.Run(result);
 
   // Trigger the pending registration.
   DCHECK(register_callbacks_.find(app_id) != register_callbacks_.end());
   RegisterImpl(app_id, normalized_sender_ids);
 }
 
 }  // namespace gcm