Fix issues with token refresh in AccountReconcilor

chrome/browser/signin/google_auto_login_helper.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/signin/google_auto_login_helper.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "google_apis/gaia/gaia_auth_fetcher.h"
 #include "google_apis/gaia/gaia_constants.h"
 
 GoogleAutoLoginHelper::GoogleAutoLoginHelper(Profile* profile)
     : profile_(profile) {}
 
 GoogleAutoLoginHelper::~GoogleAutoLoginHelper() {}

[Roger Tawa OOO till Jul 10th, 2013/11/27 18:45:43]

Should DCHECK that |accounts_| is empty.

[acleung1, 2013/11/28 00:52:44]

Done.

 
 void GoogleAutoLoginHelper::LogIn() {
-  uber_token_fetcher_.reset(new UbertokenFetcher(profile_, this));
+  uber_token_fetcher_.reset(CreateNewUbertokenFetcher());

[Roger Tawa OOO till Jul 10th, 2013/11/27 18:45:43]

Add a DCHECK that |accounts_| is empty.

[acleung1, 2013/11/28 00:52:44]

Done.

   uber_token_fetcher_->StartFetchingToken();
 }
 
 void GoogleAutoLoginHelper::LogIn(const std::string& account_id) {
-  uber_token_fetcher_.reset(new UbertokenFetcher(profile_, this));
-  uber_token_fetcher_->StartFetchingToken(account_id);
+  if (uber_token_fetcher_.get()) {
+    accounts_.push_back(account_id);
+  } else {
+    uber_token_fetcher_.reset(CreateNewUbertokenFetcher());
+    uber_token_fetcher_->StartFetchingToken(account_id);
+  }
 }
 
 void GoogleAutoLoginHelper::OnUbertokenSuccess(const std::string& uber_token) {
-  gaia_auth_fetcher_.reset(new GaiaAuthFetcher(
-      this, GaiaConstants::kChromeSource, profile_->GetRequestContext()));
+  VLOG(1) << "GoogleAutoLoginHelper::OnUbertokenSuccess" << uber_token;

[Roger Tawa OOO till Jul 10th, 2013/11/27 18:45:43]

Nit: put a space after "Success" in string?

[acleung1, 2013/11/28 00:52:44]

Good point. Actually, the uber token's too noisy. I am removing it instead.

+  gaia_auth_fetcher_.reset(CreateNewGaiaAuthFetcher());
   gaia_auth_fetcher_->StartMergeSession(uber_token);
 }
 
 void GoogleAutoLoginHelper::OnUbertokenFailure(
     const GoogleServiceAuthError& error) {
   VLOG(1) << "Failed to retrieve ubertoken, error: " << error.ToString();
   base::MessageLoop::current()->DeleteSoon(FROM_HERE, this);
 }
 
 void GoogleAutoLoginHelper::OnMergeSessionSuccess(const std::string& data) {
   DVLOG(1) << "MergeSession successful." << data;
-  base::MessageLoop::current()->DeleteSoon(FROM_HERE, this);
+  if (accounts_.empty()) {
+    base::MessageLoop::current()->DeleteSoon(FROM_HERE, this);
+  } else {
+    uber_token_fetcher_.reset(CreateNewUbertokenFetcher());
+    uber_token_fetcher_->StartFetchingToken(accounts_.front());
+    accounts_.pop_front();
+  }
 }
 
 void GoogleAutoLoginHelper::OnMergeSessionFailure(
     const GoogleServiceAuthError& error) {
   VLOG(1) << "Failed MergeSession request, error: " << error.ToString();
   base::MessageLoop::current()->DeleteSoon(FROM_HERE, this);
+  // TODO(acleung): Depending on the return error we should probably
+  // take different actions instead of just throw our hands up.

[Roger Tawa OOO till Jul 10th, 2013/11/27 18:45:43]

sgtm, but for now please clear out the |accounts_| member, since we are ignoring
them any way.  Clear out at line 40 above too.

[acleung1, 2013/11/28 00:52:44]

Done.

 }
+
+UbertokenFetcher* GoogleAutoLoginHelper::CreateNewUbertokenFetcher() {
+  return new UbertokenFetcher(profile_, this);
+}
+
+GaiaAuthFetcher* GoogleAutoLoginHelper::CreateNewGaiaAuthFetcher() {
+ return new GaiaAuthFetcher(
+      this, GaiaConstants::kChromeSource, profile_->GetRequestContext());
+}
+