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

Unified Diff: trunk/src/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h

Issue 319273002: Revert 275550 "Revert 275046 "ChromeOS login webui refactoring: ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 6 months 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: trunk/src/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h
===================================================================
--- trunk/src/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h (revision 275554)
+++ trunk/src/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h (working copy)
@@ -88,6 +88,10 @@
// Show sign-in screen for the given credentials.
virtual void ShowSigninScreenForCreds(const std::string& username,
const std::string& password) = 0;
+ virtual void LoadUsers(const base::ListValue& users_list,
+ bool animated,
+ bool show_guest) = 0;
+
protected:
virtual ~LoginDisplayWebUIHandler() {}
};
@@ -166,9 +170,6 @@
// Public sessions are always shown.
virtual bool IsShowUsers() const = 0;
- // Whether new user pod is available.
- virtual bool IsShowNewUser() const = 0;
-
// Returns true if sign in is in progress.
virtual bool IsSigninInProgress() const = 0;
@@ -186,6 +187,18 @@
virtual void LoginAsKioskApp(const std::string& app_id,
bool diagnostic_mode) = 0;
+ // Request to (re)load user list.
+ virtual void HandleGetUsers() = 0;
+
+ // Set authentication type (for easier unlocking).
+ virtual void SetAuthType(
+ const std::string& username,
+ ScreenlockBridge::LockHandler::AuthType auth_type) = 0;
+
+ // Get authentication type (for easier unlocking).
+ virtual ScreenlockBridge::LockHandler::AuthType GetAuthType(
+ const std::string& username) const = 0;
+
protected:
virtual ~SigninScreenHandlerDelegate() {}
};
@@ -282,6 +295,9 @@
virtual void ShowErrorScreen(LoginDisplay::SigninError error_id) OVERRIDE;
virtual void ShowSigninScreenForCreds(const std::string& username,
const std::string& password) OVERRIDE;
+ virtual void LoadUsers(const base::ListValue& users_list,
+ bool animated,
+ bool show_guest) OVERRIDE;
// ui::EventHandler implementation:
virtual void OnKeyEvent(ui::KeyEvent* key) OVERRIDE;
@@ -370,17 +386,6 @@
void HandleLaunchKioskApp(const std::string& app_id, bool diagnostic_mode);
void HandleRetrieveAuthenticatedUserEmail(double attempt_token);
- // Fills |user_dict| with information about |user|.
- static void FillUserDictionary(
- User* user,
- bool is_owner,
- bool is_signin_to_add,
- ScreenlockBridge::LockHandler::AuthType auth_type,
- base::DictionaryValue* user_dict);
-
- // Sends user list to account picker.
- void SendUserList(bool animated);
-
// Kick off cookie / local storage cleanup.
void StartClearingCookies(const base::Closure& on_clear_callback);
void OnCookiesCleared(base::Closure on_clear_callback);
@@ -529,11 +534,6 @@
base::Closure kiosk_enable_flow_aborted_callback_for_test_;
- // Map of usernames to their current authentication type. If a user is not
- // contained in the map, it is using the default authentication type.
- std::map<std::string, ScreenlockBridge::LockHandler::AuthType>
- user_auth_type_map_;
-
// Non-owning ptr.
// TODO (ygorshenin@): remove this dependency.
GaiaScreenHandler* gaia_screen_handler_;

Powered by Google App Engine
This is Rietveld 408576698