Chromium Code Reviews| Index: chrome/browser/signin/easy_unlock_service_signin_chromeos.h |
| diff --git a/chrome/browser/signin/easy_unlock_service_signin_chromeos.h b/chrome/browser/signin/easy_unlock_service_signin_chromeos.h |
| index 8dd90f36e7c38f0079fad5ce3db37ead568b173d..7d1b8695ea13601c12777758a5a5a1d5460e6bc0 100644 |
| --- a/chrome/browser/signin/easy_unlock_service_signin_chromeos.h |
| +++ b/chrome/browser/signin/easy_unlock_service_signin_chromeos.h |
| @@ -31,17 +31,18 @@ class EasyUnlockServiceSignin |
| explicit EasyUnlockServiceSignin(Profile* profile); |
| ~EasyUnlockServiceSignin() override; |
| - // Sets |user_id| as the current user of the service. Note this does |
| + // Sets |account_id| as the current user of the service. Note this does |
| // not change the focused user on the login screen. |
| - void SetCurrentUser(const std::string& user_id); |
| + void SetCurrentUser(const AccountId& account_id); |
| - // Wraps the challenge for the remote device identified by |user_id| and the |
| + // Wraps the challenge for the remote device identified by |account_id| and |
| + // the |
| // |device_public_key|. The |channel_binding_data| is signed by the TPM |
| // included in the wrapped challenge. |
| // |callback| will be invoked when wrapping is complete. If the user data is |
| // not loaded yet, then |callback| will be invoked with an empty string. |
| void WrapChallengeForUserAndDevice( |
| - const std::string& user_id, |
| + const AccountId& account_id, |
| const std::string& device_public_key, |
| const std::string& channel_binding_data, |
| base::Callback<void(const std::string& wraped_challenge)> callback); |
| @@ -78,7 +79,7 @@ class EasyUnlockServiceSignin |
| // EasyUnlockService implementation: |
| EasyUnlockService::Type GetType() const override; |
| - std::string GetUserEmail() const override; |
| + const AccountId GetAccountId() const override; |
|
achuithb
2015/12/04 10:12:53
AccountId
Alexander Alekseev
2015/12/04 12:44:07
Done.
|
| void LaunchSetup() override; |
| const base::DictionaryValue* GetPermitAccess() const override; |
| void SetPermitAccess(const base::DictionaryValue& permit) override; |
| @@ -91,9 +92,9 @@ class EasyUnlockServiceSignin |
| TurnOffFlowStatus GetTurnOffFlowStatus() const override; |
| std::string GetChallenge() const override; |
| std::string GetWrappedSecret() const override; |
| - void RecordEasySignInOutcome(const std::string& user_id, |
| + void RecordEasySignInOutcome(const AccountId& account_id, |
| bool success) const override; |
| - void RecordPasswordLoginEvent(const std::string& user_id) const override; |
| + void RecordPasswordLoginEvent(const AccountId& account_id) const override; |
| void StartAutoPairing(const AutoPairingResultCallback& callback) override; |
| void SetAutoPairingResult(bool success, const std::string& error) override; |
| void InitializeInternal() override; |
| @@ -108,7 +109,7 @@ class EasyUnlockServiceSignin |
| void OnScreenDidUnlock( |
| proximity_auth::ScreenlockBridge::LockHandler::ScreenType screen_type) |
| override; |
| - void OnFocusedUserChanged(const std::string& user_id) override; |
| + void OnFocusedUserChanged(const AccountId& account_id) override; |
| // chromeos::LoginState::Observer implementation: |
| void LoggedInStateChanged() override; |
| @@ -118,20 +119,19 @@ class EasyUnlockServiceSignin |
| void LoadCurrentUserDataIfNeeded(); |
| // Callback invoked when the user's device data is loaded from cryptohome. |
| - void OnUserDataLoaded( |
| - const std::string& user_id, |
| - bool success, |
| - const chromeos::EasyUnlockDeviceKeyDataList& data); |
| + void OnUserDataLoaded(const AccountId& account_id, |
| + bool success, |
| + const chromeos::EasyUnlockDeviceKeyDataList& data); |
| // If the device data has been loaded for the current user, returns it. |
| // Otherwise, returns NULL. |
| const UserData* FindLoadedDataForCurrentUser() const; |
| // User id of the user currently associated with the service. |
| - std::string user_id_; |
| + AccountId account_id_; |
| - // Maps user ids to their fetched cryptohome key data. |
| - std::map<std::string, UserData*> user_data_; |
| + // Maps account ids to their fetched cryptohome key data. |
| + std::map<AccountId, UserData*> user_data_; |
| // Whether failed attempts to load user data should be retried. |
| // This is to handle case where cryptohome daemon is not started in time the |