Update locally managed user creation flow according to mocks

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

Index: chrome/browser/ui/webui/chromeos/login/locally_managed_user_creation_screen_handler.h
diff --git a/chrome/browser/ui/webui/chromeos/login/locally_managed_user_creation_screen_handler.h b/chrome/browser/ui/webui/chromeos/login/locally_managed_user_creation_screen_handler.h
index 7993b9b0969e695a0f8f15be4da4df6801eceb16..2de0598e40aa7b5a7d50261ff94a2c12ec79e1a2 100644
--- a/chrome/browser/ui/webui/chromeos/login/locally_managed_user_creation_screen_handler.h
+++ b/chrome/browser/ui/webui/chromeos/login/locally_managed_user_creation_screen_handler.h
@@ -32,16 +32,19 @@ class LocallyManagedUserCreationScreenHandler : public BaseScreenHandler {
         LocallyManagedUserCreationScreenHandler* actor) = 0;
 
     // Starts managed user creation flow, with manager identified by
+    // |manager_id| and |manager_password|.
+    virtual void AuthenticateManager(std::string& manager_id,
+                                     std::string& manager_password) = 0;
+    // Starts managed user creation flow, with manager identified by

[ygorshenin1, 2013/04/12 12:49:35]

nit: insert an empty line before #38 and #44

[Denis Kuznetsov (DE-MUC), 2013/04/12 13:00:17]

Done.

     // |manager_id| and |manager_password|, and locally managed user that would
     // have |display_name| and authenticated by the |managed_user_password|.
-    virtual void RunFlow(string16& display_name,
-                         std::string& managed_user_password,
-                         std::string& manager_id,
-                         std::string& manager_password) = 0;
+    virtual void CreateManagedUser(string16& display_name,
+                                   std::string& managed_user_password) = 0;
+    // Starts picture selection for created managed user.
+    virtual void SelectPicture() = 0;
+
     virtual void AbortFlow() = 0;
     virtual void FinishFlow() = 0;
-    virtual void RetryLastStep() = 0;
-    virtual void SelectPicture() = 0;
   };
 
   LocallyManagedUserCreationScreenHandler();
@@ -52,13 +55,18 @@ class LocallyManagedUserCreationScreenHandler : public BaseScreenHandler {
   virtual void Hide();
   virtual void SetDelegate(Delegate* delegate);
 
-  void ShowManagerInconsistentStateErrorScreen();
   void ShowManagerPasswordError();
-  void ShowInitialScreen();
-  void ShowProgressScreen();
-  void ShowPostImageSelectionScreen();
-  virtual void ShowSuccessMessage();
-  virtual void ShowErrorMessage(string16 message, bool recoverable);
+
+  void ShowIntroPage();
+  void ShowManagerSelectionPage();
+  void ShowUsernamePage();
+  void ShowProgressPage();
+  void ShowTutorialPage();
+
+  void ShowSuccessMessage();
+
+  void ShowManagerInconsistentStateErrorPage();
+  void ShowErrorPage(string16 message, bool recoverable);

[ygorshenin1, 2013/04/12 12:49:35]

Why not to pass |message| by const ref?

[Denis Kuznetsov (DE-MUC), 2013/04/12 13:00:17]

Done.

 
   // BaseScreenHandler implementation:
   virtual void GetLocalizedStrings(base::DictionaryValue* localized_strings)
@@ -70,12 +78,18 @@ class LocallyManagedUserCreationScreenHandler : public BaseScreenHandler {
 
  private:
   // WebUI message handlers.
+  void HandleCheckLocallyManagedUserName(const base::ListValue* args);
+
+  void HandleManagerSelected(const base::ListValue* args);
+
   void HandleFinishLocalManagedUserCreation(const base::ListValue* args);
   void HandleAbortLocalManagedUserCreation(const base::ListValue* args);
   void HandleRetryLocalManagedUserCreation(const base::ListValue* args);
 
-  void HandleCheckLocallyManagedUserName(const base::ListValue* args);
-  void HandleRunLocallyManagedUserCreationFlow(const base::ListValue* args);
+  void HandleAuthenticateManager(const base::ListValue* args);
+  void HandleCreateManagedUser(const base::ListValue* args);
+
+  void UpdateText(const std::string& element_id, const string16& text);
 
   Delegate* delegate_;