Push API: Fix unsubscribing from GCM on Android

chrome/browser/extensions/extension_gcm_app_handler.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 "chrome/browser/extensions/extension_gcm_app_handler.h"
 
 #include "base/bind.h"
 #include "base/lazy_instance.h"
 #include "base/location.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 114 matching lines @@
   }
 
   RemoveAppHandler(extension->id());
 }
 
 void ExtensionGCMAppHandler::OnExtensionUninstalled(
     content::BrowserContext* browser_context,
     const Extension* extension,
     extensions::UninstallReason reason) {
   if (IsGCMPermissionEnabled(extension)) {
+#if defined(OS_ANDROID)
+    LOG(FATAL) << "Unregistering from GCM requires a sender_id on Android";
+#else
+    std::vector<std::string> sender_ids;  // These are ignored on desktop.
     GetGCMDriver()->Unregister(
         extension->id(),
+        sender_ids,
         base::Bind(&ExtensionGCMAppHandler::OnUnregisterCompleted,
                    weak_factory_.GetWeakPtr(),
                    extension->id()));
     RemoveAppHandler(extension->id());
+#endif
   }
 }
 
 void ExtensionGCMAppHandler::AddDummyAppHandler() {
   AddAppHandler(kDummyAppId);
 }
 
 void ExtensionGCMAppHandler::RemoveDummyAppHandler() {
   RemoveAppHandler(kDummyAppId);
 }
 
 gcm::GCMDriver* ExtensionGCMAppHandler::GetGCMDriver() const {
   return gcm::GCMProfileServiceFactory::GetForProfile(profile_)->driver();
 }
 
 void ExtensionGCMAppHandler::OnUnregisterCompleted(
     const std::string& app_id, gcm::GCMClient::Result result) {
   // Nothing to do.
 }
 
 void ExtensionGCMAppHandler::AddAppHandler(const std::string& app_id) {
   GetGCMDriver()->AddAppHandler(app_id, this);
 }
 
 void ExtensionGCMAppHandler::RemoveAppHandler(const std::string& app_id) {
   GetGCMDriver()->RemoveAppHandler(app_id);
 }
 
 }  // namespace extensions