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

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

Issue 8395042: [cros,de-hack] Get rid of singleton for the WebUILoginScreen. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: comment nit Created 9 years, 2 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 5a474a805520e7be74c81e4533540d3c9b1e8502..ab375a618403eb30c6da07369c99f9638bdd3dce 100644
--- a/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h
+++ b/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h
@@ -13,6 +13,7 @@
#include "base/task.h"
#include "chrome/browser/browsing_data_remover.h"
#include "chrome/browser/chromeos/login/help_app_launcher.h"
+#include "chrome/browser/chromeos/login/user_manager.h"
#include "chrome/browser/chromeos/system_key_event_listener.h"
#include "chrome/browser/ui/webui/chromeos/login/base_screen_handler.h"
#include "content/browser/webui/web_ui.h"
@@ -76,6 +77,15 @@ class SigninScreenHandlerDelegate {
// Let the delegate know about the handler it is supposed to be using.
virtual void SetWebUIHandler(LoginDisplayWebUIHandler* webui_handler) = 0;
+ // Returns users list to be shown.
+ virtual const std::vector<UserManager::User>& GetUsers() const = 0;
+
+ // Whether login as guest is available.
+ virtual bool IsShowGuest() const = 0;
+
+ // Whether new user pod is available.
+ virtual bool IsShowNewUser() const = 0;
+
protected:
virtual ~SigninScreenHandlerDelegate() {}
};
@@ -94,6 +104,10 @@ class SigninScreenHandler : public BaseScreenHandler,
// screen is for OOBE or usual sign-in flow.
void Show(bool oobe_ui);
+ // Sets delegate to be used by the handler. It is guaranteed that valid
+ // delegate is set before Show() method will be called.
+ void SetDelegate(SigninScreenHandlerDelegate* delegate);
+
private:
friend class ReportDnsCacheClearedOnUIThread;
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/oobe_ui.cc ('k') | chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698