[ChromeOS] WebUI OOBE/Login refactoring.

chrome/browser/ui/webui/chromeos/login/login_ui.cc

Index: chrome/browser/ui/webui/chromeos/login/login_ui.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/login_ui.cc b/chrome/browser/ui/webui/chromeos/login/login_ui.cc
index 3cca35654393d58e15d17b0c891008c78e37f3fa..06723cf1447ea6b9918b038117d7e7346fbe8430 100644
--- a/chrome/browser/ui/webui/chromeos/login/login_ui.cc
+++ b/chrome/browser/ui/webui/chromeos/login/login_ui.cc
@@ -5,37 +5,43 @@
 #include "chrome/browser/ui/webui/chromeos/login/login_ui.h"
 
 #include "base/memory/ref_counted_memory.h"
-#include "base/memory/singleton.h"
-#include "base/string_piece.h"
-#include "base/stringprintf.h"
-#include "base/utf_string_conversions.h"
 #include "base/values.h"
-#include "chrome/browser/chromeos/cros/cros_library.h"
-#include "chrome/browser/chromeos/cros/power_library.h"
-#include "chrome/browser/chromeos/login/user_manager.h"
-#include "chrome/browser/chromeos/login/webui_login_display.h"
 #include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/webui/chrome_url_data_manager.h"
 #include "chrome/browser/ui/webui/chromeos/login/login_ui_helpers.h"
+#include "chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h"
 #include "chrome/browser/ui/webui/options/chromeos/user_image_source.h"
 #include "chrome/browser/ui/webui/theme_source.h"
 #include "chrome/common/url_constants.h"
-#include "content/browser/browser_thread.h"
 #include "content/browser/tab_contents/tab_contents.h"
-#include "grit/generated_resources.h"
-#include "grit/theme_resources.h"
-#include "ui/base/l10n/l10n_util.h"
-#include "ui/base/resource/resource_bundle.h"
 
 namespace chromeos {
 
+// Boilerplate class that is used to associate the LoginUI code with the URL
+// "chrome://login"
+class LoginUIHTMLSource : public ChromeURLDataManager::DataSource {
+ public:
+  explicit LoginUIHTMLSource(base::DictionaryValue* localized_strings);
+  virtual ~LoginUIHTMLSource();
+
+  virtual void StartDataRequest(const std::string& path,
+                                bool is_incognito,
+                                int request_id);
+  virtual std::string GetMimeType(const std::string&) const;
+
+ private:
+  scoped_ptr<HTMLOperationsInterface> html_operations_;
+  scoped_ptr<DictionaryValue> localized_strings_;
+
+  DISALLOW_COPY_AND_ASSIGN(LoginUIHTMLSource);
+};
+
 // LoginUIHTMLSource, public: --------------------------------------------------
 
-LoginUIHTMLSource::LoginUIHTMLSource(MessageLoop* message_loop)
-    : DataSource(chrome::kChromeUILoginHost, message_loop),
-      html_operations_(new HTMLOperationsInterface()) {
+LoginUIHTMLSource::LoginUIHTMLSource(base::DictionaryValue* localized_strings)
+    : DataSource(chrome::kChromeUILoginHost, MessageLoop::current()),
+      html_operations_(new HTMLOperationsInterface()),
+      localized_strings_(localized_strings) {
 }
 
 LoginUIHTMLSource::~LoginUIHTMLSource() {
@@ -44,12 +50,11 @@ LoginUIHTMLSource::~LoginUIHTMLSource() {
 void LoginUIHTMLSource::StartDataRequest(const std::string& path,
                                          bool is_incognito,
                                          int request_id) {
-  DictionaryValue localized_strings;
-  SetFontAndTextDirection(&localized_strings);
+  SetFontAndTextDirection(localized_strings_.get());
 
   base::StringPiece login_html = html_operations_->GetLoginHTML();
-  std::string full_html = html_operations_->GetFullHTML(login_html,
-                                                        &localized_strings);
+  std::string full_html = html_operations_->GetFullHTML(
+      login_html, localized_strings_.get());
 
   scoped_refptr<RefCountedBytes> html_bytes(
       html_operations_->CreateHTMLBytes(full_html));
@@ -61,146 +66,19 @@ std::string LoginUIHTMLSource::GetMimeType(const std::string&) const {
   return "text/html";
 }
 
-// LoginUIHandlerDelegate, public: ------------------------------------------
-
-void LoginUIHandlerDelegate::set_login_handler(
-    BaseLoginUIHandler* login_handler) {
-  login_handler_ = login_handler;
-}
-
-// LoginUIHandlerDelegate, protected: ------------------------------------------
-
-LoginUIHandlerDelegate::~LoginUIHandlerDelegate() {}
-
-// LoginUIHandler, public: -----------------------------------------------------
-
-LoginUIHandler::LoginUIHandler() {
-  delegate_ = WebUILoginDisplay::GetInstance();
-  delegate_->set_login_handler(this);
-}
-
-LoginUIHandler::~LoginUIHandler() {
-}
-
-WebUIMessageHandler* LoginUIHandler::Attach(WebUI* web_ui) {
-  return WebUIMessageHandler::Attach(web_ui);
-}
-
-void LoginUIHandler::RegisterMessages() {
-  web_ui_->RegisterMessageCallback(
-      "LaunchIncognito",
-      NewCallback(this,
-                  &LoginUIHandler::HandleLaunchIncognito));
-  web_ui_->RegisterMessageCallback(
-      "AuthenticateUser",
-      NewCallback(this,
-                  &LoginUIHandler::HandleAuthenticateUser));
-  web_ui_->RegisterMessageCallback(
-      "ShutdownSystem",
-      NewCallback(this,
-                  &LoginUIHandler::HandleShutdownSystem));
-  web_ui_->RegisterMessageCallback(
-      "GetUsers",
-      NewCallback(this,
-                  &LoginUIHandler::HandleGetUsers));
-  web_ui_->RegisterMessageCallback(
-      "RemoveUser",
-      NewCallback(this,
-                  &LoginUIHandler::HandleRemoveUser));
-}
-
-void LoginUIHandler::HandleGetUsers(const ListValue* args) {
-  ListValue get_users_args;
-
-  // Grab the users from the user manager.
-  UserVector users = UserManager::Get()->GetUsers();
-  for (UserVector::const_iterator it = users.begin();
-       it != users.end(); ++it) {
-    const std::string& email = it->email();
-    DictionaryValue* user_dict = new DictionaryValue();
-    user_dict->SetString("name", it->GetDisplayName());
-    user_dict->SetString("email_address", email);
-    user_dict->SetBoolean("can_remove", !email.empty());
-    if (!email.empty()) {
-      long long timestamp = base::TimeTicks::Now().ToInternalValue();
-      std::string image_url(
-          StringPrintf("%s%s?id=%lld",
-                      chrome::kChromeUIUserImageURL,
-                      email.c_str(),
-                      timestamp));
-      user_dict->SetString("image_url", image_url);
-    } else {
-      std::string image_url(std::string(chrome::kChromeUIScheme) + "://" +
-          std::string(chrome::kChromeUIThemePath) + "/IDR_LOGIN_DEFAULT_USER");
-      user_dict->SetString("image_url", image_url);
-    }
-    get_users_args.Append(user_dict);
-  }
-
-  // Add the Guest to the user list.
-  DictionaryValue* guest_dict = new DictionaryValue();
-  guest_dict->SetString("name", l10n_util::GetStringUTF16(IDS_GUEST));
-  guest_dict->SetString("email_address", "");
-  guest_dict->SetBoolean("can_remove", false);
-  std::string image_url(std::string(chrome::kChromeUIScheme) + "://" +
-      std::string(chrome::kChromeUIThemePath) + "/IDR_LOGIN_GUEST");
-  guest_dict->SetString("image_url", image_url);
-  get_users_args.Append(guest_dict);
-
-  // Call the Javascript callback
-  web_ui_->CallJavascriptFunction("getUsersCallback", get_users_args);
-}
-
-// LoginUIHandler, private: ----------------------------------------------------
-
-void LoginUIHandler::ClearAndEnablePassword() {
-  web_ui_->CallJavascriptFunction("resetPrompt");
-}
-
-void LoginUIHandler::ShowError(const std::string& error_text,
-                               const std::string& help_link_text,
-                               HelpAppLauncher::HelpTopic help_topic_id) {
-  // TODO(xiyuan): Pass error + help to a propery error UI and save topic id.
-  ClearAndEnablePassword();
-}
-
-void LoginUIHandler::HandleAuthenticateUser(const ListValue* args) {
-  std::string username;
-  std::string password;
-  if (!args->GetString(0, &username) ||
-      !args->GetString(1, &password)) {
-    NOTREACHED();
-    return;
-  }
-
-  delegate_->Login(username, password);
-}
-
-void LoginUIHandler::HandleLaunchIncognito(const ListValue* args) {
-  delegate_->LoginAsGuest();
-}
-
-void LoginUIHandler::HandleShutdownSystem(const ListValue* args) {
-  DCHECK(CrosLibrary::Get()->EnsureLoaded());
-  CrosLibrary::Get()->GetPowerLibrary()->RequestShutdown();
-}
-
-void LoginUIHandler::HandleRemoveUser(const ListValue* args) {
-  std::string email;
-  DCHECK_EQ(args->GetSize(), 1UL);
-  args->GetString(0, &email);
-  UserManager::Get()->RemoveUserFromList(email);
-}
-
 // LoginUI, public: ------------------------------------------------------------
 
 LoginUI::LoginUI(TabContents* contents)
     : ChromeWebUI(contents) {
-  LoginUIHandler* handler = new LoginUIHandler();
-  AddMessageHandler(handler->Attach(this));
-  LoginUIHTMLSource* html_source =
-      new LoginUIHTMLSource(MessageLoop::current());
+  scoped_ptr<base::DictionaryValue> localized_strings(
+      new base::DictionaryValue);
 
+  BaseScreenHandler* signin_screen_handler = new SigninScreenHandler;
+  AddMessageHandler(signin_screen_handler->Attach(this));
+  signin_screen_handler->GetLocalizedStrings(localized_strings.get());
+
+  LoginUIHTMLSource* html_source =
+      new LoginUIHTMLSource(localized_strings.release());
   contents->profile()->GetChromeURLDataManager()->AddDataSource(html_source);
 
   // Load the theme URLs.