[GCM] API update to allow only a single sender in registration

chrome/browser/services/gcm/gcm_client_mock.cc

 // Copyright 2013 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/services/gcm/gcm_client_mock.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/sys_byteorder.h"
@@ skipping 42 matching lines @@
                  weak_ptr_factory_.GetWeakPtr()));
 }
 
 void GCMClientMock::CheckOut() {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
   status_ = CHECKED_OUT;
 }
 
 void GCMClientMock::Register(const std::string& app_id,
                              const std::string& cert,
-                             const std::vector<std::string>& sender_ids) {
+                             const std::string& sender_id) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
 
   std::string registration_id;
   if (error_simulation_ == ALWAYS_SUCCEED)
-    registration_id = GetRegistrationIdFromSenderIds(sender_ids);
+    registration_id = GetRegistrationIdFromSenderId(sender_id);
 
   base::MessageLoop::current()->PostTask(
       FROM_HERE,
       base::Bind(&GCMClientMock::RegisterFinished,
                  weak_ptr_factory_.GetWeakPtr(),
                  app_id,
                  registration_id));
 }
 
 void GCMClientMock::Unregister(const std::string& app_id) {
@@ skipping 39 matching lines @@
 
   content::BrowserThread::PostTask(
       content::BrowserThread::IO,
       FROM_HERE,
       base::Bind(&GCMClientMock::MessagesDeleted,
                  weak_ptr_factory_.GetWeakPtr(),
                  app_id));
 }
 
 // static
-std::string GCMClientMock::GetRegistrationIdFromSenderIds(
-    const std::vector<std::string>& sender_ids) {
-  // GCMProfileService normalizes 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());
-
-  // Simulate the registration_id by concaternating all sender IDs.
-  // Set registration_id to empty to denote an error if sender_ids contains a
+std::string GCMClientMock::GetRegistrationIdFromSenderId(
+    const std::string& sender_id) {
+  // Simulate the registration_id by appending reg_id to the sender ID.
+  // Set registration_id to empty to denote an error if sender_id contains a
   // hint.
-  std::string registration_id;
-  if (sender_ids.size() != 1 ||
-      sender_ids[0].find("error") == std::string::npos) {
-    for (size_t i = 0; i < normalized_sender_ids.size(); ++i) {
-      if (i > 0)
-        registration_id += ",";
-      registration_id += normalized_sender_ids[i];
-    }
-  }
-  return registration_id;
+  if (sender_id.find("error") == std::string::npos)
+    return sender_id + "_reg_id";
+  return std::string();
 }
 
 void GCMClientMock::CheckinFinished() {
   delegate_->OnGCMReady();
 }
 
 void GCMClientMock::RegisterFinished(const std::string& app_id,
                                      const std::string& registrion_id) {
   delegate_->OnRegisterFinished(
       app_id, registrion_id, registrion_id.empty() ? SERVER_ERROR : SUCCESS);
@@ skipping 26 matching lines @@
     delegate_->OnMessagesDeleted(app_id);
 }
 
 void GCMClientMock::MessageSendError(const std::string& app_id,
                                      const std::string& message_id) {
   if (delegate_)
     delegate_->OnMessageSendError(app_id, message_id, NETWORK_ERROR);
 }
 
 }  // namespace gcm