Place the spinner in the right corner of the controls window.

chrome/browser/chromeos/login/existing_user_controller.cc

Index: chrome/browser/chromeos/login/existing_user_controller.cc
diff --git a/chrome/browser/chromeos/login/existing_user_controller.cc b/chrome/browser/chromeos/login/existing_user_controller.cc
index 9718534e801e930ef988fbfb8f3d6144019b9299..46d0af741c6d3e96e293fe0a432a622979db0221 100644
--- a/chrome/browser/chromeos/login/existing_user_controller.cc
+++ b/chrome/browser/chromeos/login/existing_user_controller.cc
@@ -214,14 +214,11 @@ void ExistingUserController::LoginNewUser(const std::string& username,
   DCHECK(new_user->is_new_user());
   if (!new_user->is_new_user())
     return;
-  NewUserView* new_user_view = new_user->new_user_view();
-  new_user_view->SetUsername(username);

[whywhat, 2010/12/13 07:57:35]

Note: this was used to auto fill username after account creation though password
was empty (we didn't want to store it in memory to pass between screens). I
guess it's okay for now to break this and fix when we'll implement account
creation.

[altimofeev, 2010/12/13 15:42:45]

Agree.

 
   if (password.empty())
     return;
 
-  new_user_view->SetPassword(password);
-  new_user_view->Login();
+  new_user->OnLogin(username, password);
 }
 
 void ExistingUserController::SelectNewUser() {
@@ -361,6 +358,13 @@ void ExistingUserController::ActivateWizard(const std::string& screen_name) {
   // is doing an animation with our windows.
   DCHECK(!delete_scheduled_instance_);
   delete_scheduled_instance_ = this;
+
+  // Stop throbbers now to prevent throbbers be visible under the newly created
+  // windows.
+  for (size_t i = 0; i < controllers_.size(); ++i) {
+    controllers_[i]->StopThrobber();
+  }
+
   delete_timer_.Start(base::TimeDelta::FromSeconds(1), this,
                       &ExistingUserController::Delete);
 }
@@ -469,7 +473,8 @@ void ExistingUserController::ShowError(int error_id,
   // For now just ignore it because error_text contains all required information
   // for end users, developers can see details string in Chrome logs.
 
-  gfx::Rect bounds = controllers_[selected_view_index_]->GetScreenBounds();
+  gfx::Rect bounds =
+      controllers_[selected_view_index_]->GetMainInputScreenBounds();

[whywhat, 2010/12/13 07:57:35]

Maybe simply call the method GetMessageBubblePosition() if that's what the
result is used for?

[altimofeev, 2010/12/13 15:42:45]

Actually it is used to calculate the bubble position, not the position itself.

   BubbleBorder::ArrowLocation arrow;
   if (controllers_[selected_view_index_]->is_new_user()) {
     arrow = BubbleBorder::LEFT_TOP;