Eliminate usage of PO2TS::GetPrimaryAccountID() in UbertokenFetcher.

chrome/browser/signin/google_auto_login_helper.cc

Index: chrome/browser/signin/google_auto_login_helper.cc
diff --git a/chrome/browser/signin/google_auto_login_helper.cc b/chrome/browser/signin/google_auto_login_helper.cc
index 1cf1cf2ec6ea914048a4a7ed6a04ee8cf87e14bf..2014803e1fa358524b616c1cd4aa9c4a45fc7ede 100644
--- a/chrome/browser/signin/google_auto_login_helper.cc
+++ b/chrome/browser/signin/google_auto_login_helper.cc
@@ -16,13 +16,6 @@ GoogleAutoLoginHelper::~GoogleAutoLoginHelper() {
   DCHECK(accounts_.empty());
 }
 
-void GoogleAutoLoginHelper::LogIn() {
-  // This is the code path for non-mirror world.
-  uber_token_fetcher_.reset(CreateNewUbertokenFetcher());
-  uber_token_fetcher_->StartFetchingToken();
-  DCHECK(accounts_.empty());
-}
-
 void GoogleAutoLoginHelper::LogIn(const std::string& account_id) {
   if (uber_token_fetcher_.get()) {

[blundell, 2014/01/09 16:44:07]

Is there anything I can do to preserve the DCHECK that was existing above for
the non-mirror case? Is there any danger that uber_token_fetcher_ will be
non-NULL in the non-mirror case?

[Roger Tawa OOO till Jul 10th, 2014/01/10 00:08:14]

No need.  When I land https://codereview.chromium.org/110373007/, LogIn() will
be implemented by calling LogIn(string), so DCHECK is gone.  That DCHCK
essentially protects the reassignment to |uber_token_fetcher_|, which would
break if done concurrently with  a call to calling LogIn(string).

     accounts_.push_back(account_id);