Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(573)

Unified Diff: components/proximity_auth/proximity_auth_system.h

Issue 1494153002: This CL replaces e-mail with AccountId in easy signin code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Bugfix in original easy unlock code' Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/proximity_auth/DEPS ('k') | components/proximity_auth/proximity_auth_system.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/proximity_auth/proximity_auth_system.h
diff --git a/components/proximity_auth/proximity_auth_system.h b/components/proximity_auth/proximity_auth_system.h
index d5bd587aaf8d5815db89a0cb5f186070bcbcc80e..a4e0a7fe80b0f83deb7ce6913a4c421997b59f46 100644
--- a/components/proximity_auth/proximity_auth_system.h
+++ b/components/proximity_auth/proximity_auth_system.h
@@ -13,6 +13,7 @@
#include "components/proximity_auth/remote_device.h"
#include "components/proximity_auth/remote_device_life_cycle.h"
#include "components/proximity_auth/screenlock_bridge.h"
+#include "components/signin/core/account_id/account_id.h"
namespace proximity_auth {
@@ -41,18 +42,19 @@ class ProximityAuthSystem : public RemoteDeviceLifeCycle::Observer,
// Stops the system.
void Stop();
- // Registers a list of |remote_devices| for |user_id| that can be used for
+ // Registers a list of |remote_devices| for |account_id| that can be used for
// sign-in/unlock. If devices were previously registered for the user, then
// they will be replaced.
- void SetRemoteDevicesForUser(const std::string& user_id,
+ void SetRemoteDevicesForUser(const AccountId& account_id,
const RemoteDeviceList& remote_devices);
- // Returns the RemoteDevices registered for |user_id|. Returns an empty list
- // if no devices are registered for |user_id|.
- RemoteDeviceList GetRemoteDevicesForUser(const std::string& user_id) const;
+ // Returns the RemoteDevices registered for |account_id|. Returns an empty
+ // list
+ // if no devices are registered for |account_id|.
+ RemoteDeviceList GetRemoteDevicesForUser(const AccountId& account_id) const;
// Called when the user clicks the user pod and attempts to unlock/sign-in.
- void OnAuthAttempted(const std::string& user_id);
+ void OnAuthAttempted(const AccountId& account_id);
// Called when the system suspends.
void OnSuspend();
@@ -70,14 +72,14 @@ class ProximityAuthSystem : public RemoteDeviceLifeCycle::Observer,
ScreenlockBridge::LockHandler::ScreenType screen_type) override;
void OnScreenDidUnlock(
ScreenlockBridge::LockHandler::ScreenType screen_type) override;
- void OnFocusedUserChanged(const std::string& user_id) override;
+ void OnFocusedUserChanged(const AccountId& account_id) override;
// Resumes |remote_device_life_cycle_| after device wakes up and waits a
// timeout.
void ResumeAfterWakeUpTimeout();
- // Lists of remote devices, keyed by user id.
- std::map<std::string, RemoteDeviceList> remote_devices_map_;
+ // Lists of remote devices, keyed by user account id.
+ std::map<AccountId, RemoteDeviceList> remote_devices_map_;
// Delegate for Chrome dependent functionality.
ProximityAuthClient* proximity_auth_client_;
« no previous file with comments | « components/proximity_auth/DEPS ('k') | components/proximity_auth/proximity_auth_system.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698