| Index: components/gcm_driver/gcm_client_impl_unittest.cc | 
| diff --git a/components/gcm_driver/gcm_client_impl_unittest.cc b/components/gcm_driver/gcm_client_impl_unittest.cc | 
| index a9a2429a9d5f02b3a91ae5f7beeee61a11ec0a05..54d978e382f2ee70656cbed7bc853984a917b5ea 100644 | 
| --- a/components/gcm_driver/gcm_client_impl_unittest.cc | 
| +++ b/components/gcm_driver/gcm_client_impl_unittest.cc | 
| @@ -248,6 +248,9 @@ class GCMClientImplTest : public testing::Test, | 
| void BuildGCMClient(base::TimeDelta clock_step); | 
| void InitializeGCMClient(); | 
| void StartGCMClient(); | 
| +  void Register(const std::string& app_id, | 
| +                const std::vector<std::string>& senders); | 
| +  void Unregister(const std::string& app_id); | 
| void ReceiveMessageFromMCS(const MCSMessage& message); | 
| void ReceiveOnMessageSentToMCS( | 
| const std::string& app_id, | 
| @@ -267,11 +270,12 @@ class GCMClientImplTest : public testing::Test, | 
| const std::string& registration_id); | 
|  | 
| // GCMClient::Delegate overrides (for verification). | 
| -  void OnRegisterFinished(const std::string& app_id, | 
| +  void OnRegisterFinished(const linked_ptr<RegistrationInfo>& registration_info, | 
| const std::string& registration_id, | 
| GCMClient::Result result) override; | 
| -  void OnUnregisterFinished(const std::string& app_id, | 
| -                            GCMClient::Result result) override; | 
| +  void OnUnregisterFinished( | 
| +      const linked_ptr<RegistrationInfo>& registration_info, | 
| +      GCMClient::Result result) override; | 
| void OnSendFinished(const std::string& app_id, | 
| const std::string& message_id, | 
| GCMClient::Result result) override {} | 
| @@ -494,17 +498,17 @@ void GCMClientImplTest::VerifyPendingRequestFetcherDeleted() { | 
| } | 
|  | 
| bool GCMClientImplTest::ExistsRegistration(const std::string& app_id) const { | 
| -  return gcm_client_->registrations_.count(app_id) > 0; | 
| +  return ExistsGCMRegistrationInMap(gcm_client_->registrations_, app_id); | 
| } | 
|  | 
| void GCMClientImplTest::AddRegistration( | 
| const std::string& app_id, | 
| const std::vector<std::string>& sender_ids, | 
| const std::string& registration_id) { | 
| -  linked_ptr<RegistrationInfo> registration(new RegistrationInfo); | 
| +  linked_ptr<GCMRegistrationInfo> registration(new GCMRegistrationInfo); | 
| +  registration->app_id = app_id; | 
| registration->sender_ids = sender_ids; | 
| -  registration->registration_id = registration_id; | 
| -  gcm_client_->registrations_[app_id] = registration; | 
| +  gcm_client_->registrations_[registration] = registration_id; | 
| } | 
|  | 
| void GCMClientImplTest::InitializeGCMClient() { | 
| @@ -527,6 +531,21 @@ void GCMClientImplTest::StartGCMClient() { | 
| PumpLoopUntilIdle(); | 
| } | 
|  | 
| +void GCMClientImplTest::Register(const std::string& app_id, | 
| +                                 const std::vector<std::string>& senders) { | 
| +  scoped_ptr<GCMRegistrationInfo> gcm_info(new GCMRegistrationInfo); | 
| +  gcm_info->app_id = app_id; | 
| +  gcm_info->sender_ids = senders; | 
| +  gcm_client()->Register(make_linked_ptr<RegistrationInfo>(gcm_info.release())); | 
| +} | 
| + | 
| +void GCMClientImplTest::Unregister(const std::string& app_id) { | 
| +  scoped_ptr<GCMRegistrationInfo> gcm_info(new GCMRegistrationInfo); | 
| +  gcm_info->app_id = app_id; | 
| +  gcm_client()->Unregister( | 
| +      make_linked_ptr<RegistrationInfo>(gcm_info.release())); | 
| +} | 
| + | 
| void GCMClientImplTest::ReceiveMessageFromMCS(const MCSMessage& message) { | 
| gcm_client_->recorder_.RecordConnectionInitiated(std::string()); | 
| gcm_client_->recorder_.RecordConnectionSuccess(); | 
| @@ -558,19 +577,21 @@ void GCMClientImplTest::OnMessageReceived( | 
| QuitLoop(); | 
| } | 
|  | 
| -void GCMClientImplTest::OnRegisterFinished(const std::string& app_id, | 
| -                                           const std::string& registration_id, | 
| -                                           GCMClient::Result result) { | 
| +void GCMClientImplTest::OnRegisterFinished( | 
| +    const linked_ptr<RegistrationInfo>& registration_info, | 
| +    const std::string& registration_id, | 
| +    GCMClient::Result result) { | 
| last_event_ = REGISTRATION_COMPLETED; | 
| -  last_app_id_ = app_id; | 
| +  last_app_id_ = registration_info->app_id; | 
| last_registration_id_ = registration_id; | 
| last_result_ = result; | 
| } | 
|  | 
| -void GCMClientImplTest::OnUnregisterFinished(const std::string& app_id, | 
| -                                             GCMClient::Result result) { | 
| +void GCMClientImplTest::OnUnregisterFinished( | 
| +    const linked_ptr<RegistrationInfo>& registration_info, | 
| +    GCMClient::Result result) { | 
| last_event_ = UNREGISTRATION_COMPLETED; | 
| -  last_app_id_ = app_id; | 
| +  last_app_id_ = registration_info->app_id; | 
| last_result_ = result; | 
| } | 
|  | 
| @@ -642,7 +663,7 @@ TEST_F(GCMClientImplTest, RegisterApp) { | 
|  | 
| std::vector<std::string> senders; | 
| senders.push_back("sender"); | 
| -  gcm_client()->Register(kAppId, senders); | 
| +  Register(kAppId, senders); | 
| CompleteRegistration("reg_id"); | 
|  | 
| EXPECT_EQ(REGISTRATION_COMPLETED, last_event()); | 
| @@ -657,7 +678,7 @@ TEST_F(GCMClientImplTest, DISABLED_RegisterAppFromCache) { | 
|  | 
| std::vector<std::string> senders; | 
| senders.push_back("sender"); | 
| -  gcm_client()->Register(kAppId, senders); | 
| +  Register(kAppId, senders); | 
| CompleteRegistration("reg_id"); | 
| EXPECT_TRUE(ExistsRegistration(kAppId)); | 
|  | 
| @@ -679,11 +700,11 @@ TEST_F(GCMClientImplTest, UnregisterApp) { | 
|  | 
| std::vector<std::string> senders; | 
| senders.push_back("sender"); | 
| -  gcm_client()->Register(kAppId, senders); | 
| +  Register(kAppId, senders); | 
| CompleteRegistration("reg_id"); | 
| EXPECT_TRUE(ExistsRegistration(kAppId)); | 
|  | 
| -  gcm_client()->Unregister(kAppId); | 
| +  Unregister(kAppId); | 
| CompleteUnregistration(kAppId); | 
|  | 
| EXPECT_EQ(UNREGISTRATION_COMPLETED, last_event()); | 
| @@ -698,7 +719,7 @@ TEST_F(GCMClientImplTest, UnregisterApp) { | 
| TEST_F(GCMClientImplTest, DeletePendingRequestsWhenStopping) { | 
| std::vector<std::string> senders; | 
| senders.push_back("sender"); | 
| -  gcm_client()->Register(kAppId, senders); | 
| +  Register(kAppId, senders); | 
|  | 
| gcm_client()->Stop(); | 
| VerifyPendingRequestFetcherDeleted(); | 
| @@ -1148,7 +1169,7 @@ TEST_F(GCMClientImplStartAndStopTest, DelayStart) { | 
| // Registration. | 
| std::vector<std::string> senders; | 
| senders.push_back("sender"); | 
| -  gcm_client()->Register(kAppId, senders); | 
| + Register(kAppId, senders); | 
| CompleteRegistration("reg_id"); | 
| EXPECT_EQ(GCMClientImpl::READY, gcm_client_state()); | 
|  | 
|  |