[GCM] Remove all persisted data and close connections when signing out of a profile

google_apis/gcm/gcm_client_impl.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 "google_apis/gcm/gcm_client_impl.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 253 matching lines @@
     }
   }
 }
 
 void GCMClientImpl::SetDeviceCredentialsCallback(bool success) {
   // TODO(fgorski): This is one of the signals that store needs a rebuild.
   DCHECK(success);
 }
 
 void GCMClientImpl::CheckOut() {
-  // TODO(jianli): to be implemented.
   delegate_ = NULL;
+  device_checkin_info_.Reset();
+  mcs_client_->Destroy();  // This will also destroy GCM store.

[fgorski, 2014/02/13 00:20:12]

Given the ownership of GCMStore the comment sounds like a documentation of a
side effect, could you add a TODO to the MCS client destroy, that we should move
part of the functionality to GCMClient::Destroy?

[jianli, 2014/02/13 00:32:48]

Done.

+  mcs_client_.reset();
+  checkin_request_.reset();
+  pending_registrations_.clear();
 }
 
 void GCMClientImpl::Register(const std::string& app_id,
                              const std::string& cert,
                              const std::vector<std::string>& sender_ids) {
   DCHECK_EQ(state_, READY);
   RegistrationRequest::RequestInfo request_info(
       device_checkin_info_.android_id,
       device_checkin_info_.secret,
       app_id,
@@ skipping 160 matching lines @@
   if (iter != incoming_message.data.end())
     message_id = iter->second;
   delegate->OnMessageSendError(app_id, message_id, SERVER_ERROR);
 }
 
 void GCMClientImpl::SetMCSClientForTesting(scoped_ptr<MCSClient> mcs_client) {
   mcs_client_ = mcs_client.Pass();
 }
 
 }  // namespace gcm