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

Unified Diff: chrome/browser/chromeos/login/signin_screen_controller.h

Issue 1470633002: ChromeOS: Fix removeUser button on signin screen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@462823--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Fix-GetKnownUserAccountId
Patch Set: Rebased. Created 5 years, 1 month 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
Index: chrome/browser/chromeos/login/signin_screen_controller.h
diff --git a/chrome/browser/chromeos/login/signin_screen_controller.h b/chrome/browser/chromeos/login/signin_screen_controller.h
index e341062248af8dc113ffc9e088f586f94bc4befb..fb9b9817b4532f00022fd548256c1ab4f621432c 100644
--- a/chrome/browser/chromeos/login/signin_screen_controller.h
+++ b/chrome/browser/chromeos/login/signin_screen_controller.h
@@ -14,6 +14,8 @@
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
+class AccountId;
+
namespace chromeos {
class LoginDisplayWebUIHandler;
@@ -47,18 +49,18 @@ class SignInScreenController : public user_manager::RemoveUserDelegate,
// Query to remove user with specified id.
// TODO(antrim): move to user selection screen handler.
- void RemoveUser(const std::string& user_id);
+ void RemoveUser(const AccountId& account_id);
+ private:
// user_manager::RemoveUserDelegate implementation:
- void OnBeforeUserRemoved(const std::string& username) override;
- void OnUserRemoved(const std::string& username) override;
+ void OnBeforeUserRemoved(const AccountId& account_id) override;
+ void OnUserRemoved(const AccountId& account_id) override;
// content::NotificationObserver implementation.
void Observe(int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) override;
- private:
static SignInScreenController* instance_;
OobeDisplay* oobe_display_;

Powered by Google App Engine
This is Rietveld 408576698