Fix to the GCM Account Tracker flooding on a stale refresh token

chrome/browser/services/gcm/gcm_account_tracker.cc

 // Copyright 2014 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_account_tracker.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/time/time.h"
@@ skipping 132 matching lines @@
   DCHECK(!request->GetAccountId().empty());
   DVLOG(1) << "Get token failure: " << request->GetAccountId();
 
   AccountInfos::iterator iter = account_infos_.find(request->GetAccountId());
   DCHECK(iter != account_infos_.end());
   if (iter != account_infos_.end()) {
     DCHECK(iter->second.state == GETTING_TOKEN ||
            iter->second.state == ACCOUNT_REMOVED);
     // If OnAccountSignedOut(..) was called most recently, account is kept in
     // ACCOUNT_REMOVED state.
-    if (iter->second.state == GETTING_TOKEN)
-      iter->second.state = TOKEN_NEEDED;
+    if (iter->second.state == GETTING_TOKEN) {
+      // Given the fetcher has a built in retry logic, consider this situation
+      // to be invalid refresh token, that is only fixed when user signs in.
+      // Once the users signs in properly the minting will retry.
+      iter->second.access_token.clear();
+      iter->second.state = ACCOUNT_REMOVED;
+    }
   }
 
   DeleteTokenRequest(request);
   ReportTokens();
 }
 
 void GCMAccountTracker::OnConnected(const net::IPEndPoint& ip_endpoint) {
   if (IsTokenReportingRequired())
     ReportTokens();
 }
@@ skipping 173 matching lines @@
   iter->second.state = ACCOUNT_REMOVED;
   ReportTokens();
 }
 
 OAuth2TokenService* GCMAccountTracker::GetTokenService() {
   DCHECK(account_tracker_->identity_provider());
   return account_tracker_->identity_provider()->GetTokenService();
 }
 
 }  // namespace gcm