[ChromeOS] WebUI OOBE/Login refactoring.

chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_SIGNIN_SCREEN_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_SIGNIN_SCREEN_HANDLER_H_
 #pragma once
 
 #include "chrome/browser/ui/webui/chromeos/login/login_ui.h"
-#include "chrome/browser/ui/webui/chromeos/login/oobe_ui.h"
+#include "chrome/browser/ui/webui/chromeos/login/base_screen_handler.h"
 #include "content/browser/webui/web_ui.h"
 
 namespace base {
 class DictionaryValue;
 class ListValue;
 }
 
 namespace chromeos {
 
-// Signin screen handler.
-class SigninScreenHandler : public OobeMessageHandler,
-                            public BaseLoginUIHandler {
+// An interface for WebUILoginDisplay to call SigninScreenHandler.
+class LoginDisplayWebUIHandler {
+ public:
+  virtual void ClearAndEnablePassword() = 0;
+  virtual void ShowError(const std::string& error_text,
+                         const std::string& help_link_text,
+                         HelpAppLauncher::HelpTopic help_topic_id) = 0;
+};
+
+// An interface for SigninScreenHandler to call WebUILoginDisplay.

[Nikita (slow), 2011/07/15 07:11:02]

nit: It has private data members so technically it's a base class.

[xiyuan, 2011/07/15 18:26:23]

You are right. Made it an interface.

+class SigninScreenHandlerDelegate {
+ public:
+  SigninScreenHandlerDelegate()
+      : webui_handler_(NULL) { }
+
+  // Sign in using |username| and |password| specified.
+  // Used for both known and new users.
+  virtual void Login(const std::string& username,
+                     const std::string& password) = 0;
+
+  // Sign in into Guest session.
+  virtual void LoginAsGuest() = 0;
+
+  // Let the delegate know about the handler it is supposed to be using.
+  void set_webui_handler(LoginDisplayWebUIHandler* webui_handler);

[Nikita (slow), 2011/07/15 07:11:02]

nit: Setter implementation should be located in .h

[xiyuan, 2011/07/15 18:26:23]

Renamed to SetWebUIHandler and made it pure virtual.

+
+ protected:
+  // Reference to the WebUI handling layer for the login screen
+  LoginDisplayWebUIHandler* webui_handler_;
+
+  virtual ~SigninScreenHandlerDelegate();
+};
+
+// A class that handles the WebUI hooks in sign-in screen in OobeDisplay
+// and LoginDisplay.
+class SigninScreenHandler : public BaseScreenHandler,
+                            public LoginDisplayWebUIHandler {
  public:
   SigninScreenHandler();
 
   // Shows the sign in screen.
   void Show();
 
  private:
-  // OobeMessageHandler implementation:
+  // BaseScreenHandler implementation:
   virtual void GetLocalizedStrings(
       base::DictionaryValue* localized_strings) OVERRIDE;
   virtual void Initialize() OVERRIDE;
 
   // WebUIMessageHandler implementation:
   virtual void RegisterMessages() OVERRIDE;
 
   // BaseLoginUIHandler implementation.
   virtual void ClearAndEnablePassword() OVERRIDE;
   virtual void ShowError(const std::string& error_text,
                          const std::string& help_link_text,
                          HelpAppLauncher::HelpTopic help_topic_id) OVERRIDE;
 
-  // Handles authenticate user request from javascript.
+  // Callbacks from javascript.
+  void HandleGetUsers(const base::ListValue* args);
   void HandleAuthenticateUser(const base::ListValue* args);
+  void HandleLaunchIncognito(const base::ListValue* args);
+  void HandleShutdownSystem(const base::ListValue* args);
+  void HandleRemoveUser(const base::ListValue* args);
 
   // A delegate that glues this handler with backend LoginDisplay.
-  LoginUIHandlerDelegate* delegate_;
+  SigninScreenHandlerDelegate* delegate_;
 
-  // Keeps whether screen should be shown right after initialization.
+  // Whether screen should be shown right after initialization.
   bool show_on_init_;
 
   DISALLOW_COPY_AND_ASSIGN(SigninScreenHandler);
 };
 
 }  // namespae chromeoc
 
 #endif  // CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_SIGNIN_SCREEN_HANDLER_H_