Replace SetAuthenticatedUsername with SetAuthenticatedAccountId.

components/signin/core/browser/account_tracker_service.cc

Index: components/signin/core/browser/account_tracker_service.cc
diff --git a/components/signin/core/browser/account_tracker_service.cc b/components/signin/core/browser/account_tracker_service.cc
index 2da5a98656f6a141fe48a960822442321be68d20..fc81b26be0d76676720593eda20d7fe937279486 100644
--- a/components/signin/core/browser/account_tracker_service.cc
+++ b/components/signin/core/browser/account_tracker_service.cc
@@ -584,12 +584,8 @@ std::string AccountTrackerService::PickAccountIdForAccount(
   }
 }
 
-void AccountTrackerService::SeedAccountInfo(const std::string& gaia,
-                                            const std::string& email) {
-  DVLOG(1) << "AccountTrackerService::SeedAccountInfo"
-           << " gaia_id=" << gaia
-           << " email=" << email;
-
+std::string AccountTrackerService::SeedAccountInfo(const std::string& gaia,
+                                                   const std::string& email) {
   DCHECK(!gaia.empty());
   DCHECK(!email.empty());
   const std::string account_id = PickAccountIdForAccount(gaia, email);
@@ -600,4 +596,11 @@ void AccountTrackerService::SeedAccountInfo(const std::string& gaia,
   state.info.gaia = gaia;
   state.info.email = email;
   SaveToPrefs(state);
+
+  DVLOG(1) << "AccountTrackerService::SeedAccountInfo"
+           << " account_id=" << account_id
+           << " gaia_id=" << gaia
+           << " email=" << email;
+
+  return account_id;

[Mike Lerman, 2015/04/08 14:45:27]

Thought: Is it worth setting a class variable in the AccountTrackerService
called "picked_account_id_" that's set after SeedAccountInfo is called, such
that calls to PickAccountIdForAccount and SeedAccountInfo can return a const
std::string&, and make PickAccountIdForAccount more efficient?

Might not be worth the code to save a few operations, but just a thought.

[Roger Tawa OOO till Jul 10th, 2015/04/08 20:24:21]

I don't think it's worth it.  SeedAccountInfo is called only during the sign in
process, which means it is very rarely called.

 }