Push API: Fix unsubscribing from GCM on Android

components/gcm_driver/gcm_driver_android.cc

Index: components/gcm_driver/gcm_driver_android.cc
diff --git a/components/gcm_driver/gcm_driver_android.cc b/components/gcm_driver/gcm_driver_android.cc
index 93b2f919ba32db7106a503b8fade0ae7c2cdcb6d..524abb0000c8b33bacf98e03c491fe985e32e88d 100644
--- a/components/gcm_driver/gcm_driver_android.cc
+++ b/components/gcm_driver/gcm_driver_android.cc
@@ -169,20 +169,23 @@ GCMClient::Result GCMDriverAndroid::EnsureStarted(
 }
 
 void GCMDriverAndroid::RegisterImpl(
-    const std::string& app_id,
-    const std::vector<std::string>& sender_ids) {
+    const std::string& app_id, const std::vector<std::string>& sender_ids) {
+  CHECK(sender_ids.size() == 1);

[jianli, 2015/02/10 23:19:25]

Is web push messaging the only client for Android? Do we have other cases
multiple sender IDs could be used on Android?

nit: use CHECK_EQ

[johnme, 2015/02/12 19:15:25]

Push API is the only client I can find for GCMDriverAndroid. In any case, the
only GCMDriver client that uses multiple sender IDs is the chrome.gcm
Apps/Extensions API, so this should be fine.
Done (and below).

   JNIEnv* env = AttachCurrentThread();
   Java_GCMDriver_register(
       env, java_ref_.obj(),
       ConvertUTF8ToJavaString(env, app_id).Release(),
-      ToJavaArrayOfStrings(env, sender_ids).obj());
+      ConvertUTF8ToJavaString(env, sender_ids[0]).Release());
 }
 
-void GCMDriverAndroid::UnregisterImpl(const std::string& app_id) {
+void GCMDriverAndroid::UnregisterImpl(
+    const std::string& app_id, const std::vector<std::string>& sender_ids) {
+  CHECK(sender_ids.size() == 1);
   JNIEnv* env = AttachCurrentThread();
   Java_GCMDriver_unregister(
       env, java_ref_.obj(),
-      ConvertUTF8ToJavaString(env, app_id).Release());
+      ConvertUTF8ToJavaString(env, app_id).Release(),
+      ConvertUTF8ToJavaString(env, sender_ids[0]).Release());
 }
 
 void GCMDriverAndroid::SendImpl(const std::string& app_id,