Index: chrome/browser/signin/easy_unlock_service_signin_chromeos.cc |
diff --git a/chrome/browser/signin/easy_unlock_service_signin_chromeos.cc b/chrome/browser/signin/easy_unlock_service_signin_chromeos.cc |
index ad3536f856e0c06e985c3d3a5f3ae6af37c9bdf6..2ee6cb89f874739ef32574395adb06e64b6e7c6b 100644 |
--- a/chrome/browser/signin/easy_unlock_service_signin_chromeos.cc |
+++ b/chrome/browser/signin/easy_unlock_service_signin_chromeos.cc |
@@ -38,7 +38,7 @@ uint32 GetNextBackoffInterval(uint32 backoff) { |
} |
void LoadDataForUser( |
- const std::string& user_id, |
+ const user_manager::UserID& user_id, |
uint32 backoff_ms, |
const chromeos::EasyUnlockKeyManager::GetDeviceDataListCallback& callback); |
@@ -49,7 +49,7 @@ void LoadDataForUser( |
// |LoadDataForUser| call with some backoff. If no further retires are allowed, |
// it invokes |callback| with the |LoadDataForUser| results. |
void RetryDataLoadOnError( |
- const std::string& user_id, |
+ const user_manager::UserID& user_id, |
uint32 backoff_ms, |
const chromeos::EasyUnlockKeyManager::GetDeviceDataListCallback& callback, |
bool success, |
@@ -73,7 +73,7 @@ void RetryDataLoadOnError( |
// Loads device data list associated with the user's Easy unlock keys. |
void LoadDataForUser( |
- const std::string& user_id, |
+ const user_manager::UserID& user_id, |
uint32 backoff_ms, |
const chromeos::EasyUnlockKeyManager::GetDeviceDataListCallback& callback) { |
chromeos::EasyUnlockKeyManager* key_manager = |
@@ -95,6 +95,7 @@ EasyUnlockServiceSignin::UserData::~UserData() {} |
EasyUnlockServiceSignin::EasyUnlockServiceSignin(Profile* profile) |
: EasyUnlockService(profile), |
+ user_id_(std::string(), std::string()), |
allow_cryptohome_backoff_(true), |
service_active_(false), |
user_pod_last_focused_timestamp_(base::TimeTicks::Now()), |
@@ -104,7 +105,7 @@ EasyUnlockServiceSignin::EasyUnlockServiceSignin(Profile* profile) |
EasyUnlockServiceSignin::~EasyUnlockServiceSignin() { |
} |
-void EasyUnlockServiceSignin::SetCurrentUser(const std::string& user_id) { |
+void EasyUnlockServiceSignin::SetCurrentUser(const user_manager::UserID& user_id) { |
OnFocusedUserChanged(user_id); |
} |
@@ -112,7 +113,7 @@ EasyUnlockService::Type EasyUnlockServiceSignin::GetType() const { |
return EasyUnlockService::TYPE_SIGNIN; |
} |
-std::string EasyUnlockServiceSignin::GetUserEmail() const { |
+user_manager::UserID EasyUnlockServiceSignin::GetUserID() const { |
return user_id_; |
} |
@@ -177,9 +178,9 @@ std::string EasyUnlockServiceSignin::GetWrappedSecret() const { |
} |
void EasyUnlockServiceSignin::RecordEasySignInOutcome( |
- const std::string& user_id, |
+ const user_manager::UserID& user_id, |
bool success) const { |
- DCHECK_EQ(GetUserEmail(), user_id); |
+ DCHECK(GetUserID() == user_id); |
RecordEasyUnlockSigninEvent( |
success ? EASY_UNLOCK_SUCCESS : EASY_UNLOCK_FAILURE); |
@@ -191,10 +192,10 @@ void EasyUnlockServiceSignin::RecordEasySignInOutcome( |
} |
void EasyUnlockServiceSignin::RecordPasswordLoginEvent( |
- const std::string& user_id) const { |
+ const user_manager::UserID& user_id) const { |
// This happens during tests, where a user could log in without the user pod |
// being focused. |
- if (GetUserEmail() != user_id) |
+ if (GetUserID() != user_id) |
return; |
if (!IsEnabled()) |
@@ -284,7 +285,7 @@ void EasyUnlockServiceSignin::OnScreenDidUnlock( |
Shutdown(); |
} |
-void EasyUnlockServiceSignin::OnFocusedUserChanged(const std::string& user_id) { |
+void EasyUnlockServiceSignin::OnFocusedUserChanged(const user_manager::UserID& user_id) { |
if (user_id_ == user_id) |
return; |
@@ -330,7 +331,7 @@ void EasyUnlockServiceSignin::LoadCurrentUserDataIfNeeded() { |
if (user_id_.empty() || !service_active_) |
return; |
- std::map<std::string, UserData*>::iterator it = user_data_.find(user_id_); |
+ std::map<user_manager::UserID, UserData*>::iterator it = user_data_.find(user_id_); |
if (it == user_data_.end()) |
user_data_.insert(std::make_pair(user_id_, new UserData())); |
@@ -349,7 +350,7 @@ void EasyUnlockServiceSignin::LoadCurrentUserDataIfNeeded() { |
} |
void EasyUnlockServiceSignin::OnUserDataLoaded( |
- const std::string& user_id, |
+ const user_manager::UserID& user_id, |
bool success, |
const chromeos::EasyUnlockDeviceKeyDataList& devices) { |
allow_cryptohome_backoff_ = false; |
@@ -385,7 +386,7 @@ const EasyUnlockServiceSignin::UserData* |
if (user_id_.empty()) |
return NULL; |
- std::map<std::string, UserData*>::const_iterator it = |
+ std::map<user_manager::UserID, UserData*>::const_iterator it = |
user_data_.find(user_id_); |
if (it == user_data_.end()) |
return NULL; |