Commit e492815b authored by James Hawkins's avatar James Hawkins Committed by Commit Bot

Smart Lock: Remove unused account ID param.

Bug: none
Test: none
Change-Id: Ica8c4e209e5969cad312ff161d5f04787a3aa0ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932701
Commit-Queue: Ryan Hansberry <hansberry@chromium.org>
Auto-Submit: James Hawkins <jhawkins@chromium.org>
Reviewed-by: default avatarRyan Hansberry <hansberry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#718713}
parent 58601a48
...@@ -384,7 +384,7 @@ void EasyUnlockService::AttemptAuth(const AccountId& account_id) { ...@@ -384,7 +384,7 @@ void EasyUnlockService::AttemptAuth(const AccountId& account_id) {
// attempted. However, we ideally should refactor the auth attempt logic to // attempted. However, we ideally should refactor the auth attempt logic to
// the proximity_auth component. // the proximity_auth component.
if (proximity_auth_system_) if (proximity_auth_system_)
proximity_auth_system_->OnAuthAttempted(account_id); proximity_auth_system_->OnAuthAttempted();
} }
void EasyUnlockService::FinalizeUnlock(bool success) { void EasyUnlockService::FinalizeUnlock(bool success) {
......
...@@ -83,8 +83,7 @@ ProximityAuthSystem::GetRemoteDevicesForUser( ...@@ -83,8 +83,7 @@ ProximityAuthSystem::GetRemoteDevicesForUser(
return remote_devices_map_.at(account_id); return remote_devices_map_.at(account_id);
} }
void ProximityAuthSystem::OnAuthAttempted(const AccountId& /* account_id */) { void ProximityAuthSystem::OnAuthAttempted() {
// TODO(tengs): There is no reason to pass the |account_id| argument anymore.
unlock_manager_->OnAuthAttempted(mojom::AuthType::USER_CLICK); unlock_manager_->OnAuthAttempted(mojom::AuthType::USER_CLICK);
} }
......
...@@ -63,7 +63,7 @@ class ProximityAuthSystem : public ScreenlockBridge::Observer { ...@@ -63,7 +63,7 @@ class ProximityAuthSystem : public ScreenlockBridge::Observer {
const AccountId& account_id) const; const AccountId& account_id) const;
// Called when the user clicks the user pod and attempts to unlock/sign-in. // Called when the user clicks the user pod and attempts to unlock/sign-in.
void OnAuthAttempted(const AccountId& account_id); void OnAuthAttempted();
// Called when the system suspends. // Called when the system suspends.
void OnSuspend(); void OnSuspend();
......
...@@ -397,7 +397,7 @@ TEST_F(ProximityAuthSystemTest, StopSystem_RegisteredUserFocused) { ...@@ -397,7 +397,7 @@ TEST_F(ProximityAuthSystemTest, StopSystem_RegisteredUserFocused) {
TEST_F(ProximityAuthSystemTest, OnAuthAttempted) { TEST_F(ProximityAuthSystemTest, OnAuthAttempted) {
FocusUser(kUser1); FocusUser(kUser1);
EXPECT_CALL(*unlock_manager_, OnAuthAttempted(_)); EXPECT_CALL(*unlock_manager_, OnAuthAttempted(_));
proximity_auth_system_->OnAuthAttempted(AccountId::FromUserEmail(kUser1)); proximity_auth_system_->OnAuthAttempted();
} }
TEST_F(ProximityAuthSystemTest, Suspend_ScreenUnlocked) { TEST_F(ProximityAuthSystemTest, Suspend_ScreenUnlocked) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment