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

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

Issue 296823009: ChromeOS login webui refactoring: split user selection/gaia login screens. (Closed) Base URL: http://git.chromium.org/chromium/src.git@refactoring-1
Patch Set: Merge with ToT again Created 6 years, 7 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: chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h
diff --git a/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h b/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h
index d283d0f974cc154381d32b8400c8208b5384e9d7..134f911ddc5221ac6bdf2b72b9fa3ff0d5da1846 100644
--- a/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h
+++ b/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h
@@ -91,6 +91,10 @@ class LoginDisplayWebUIHandler {
// 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() {}
};
@@ -169,9 +173,6 @@ class SigninScreenHandlerDelegate {
// 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;
@@ -189,6 +190,18 @@ class SigninScreenHandlerDelegate {
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() {}
};
@@ -288,6 +301,9 @@ class SigninScreenHandler
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;
@@ -376,17 +392,6 @@ class SigninScreenHandler
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);
@@ -537,11 +542,6 @@ class SigninScreenHandler
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