Uber tray unresponsive with Multiprofile enabled

chrome/browser/chromeos/login/existing_user_controller.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/chromeos/login/existing_user_controller.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 422 matching lines @@
 
   if (!user_context.HasCredentials()) {
     // For easy unlock auth, login UI gets disabled prior to attempting login.
     if (user_context.GetAuthFlow() == UserContext::AUTH_FLOW_EASY_UNLOCK)
       login_display_->SetUIEnabled(true);
     return;
   }
 
   PerformPreLoginActions(user_context);
   PerformLogin(user_context, LoginPerformer::AUTH_MODE_INTERNAL);
+  PerformLoginFinishedActions(false);

[Nikita (slow), 2014/10/28 16:31:19]

That's not a correct place for this call since login request has just been
created - previous PerformLogin() call.

[afakhry, 2014/10/28 17:06:30]

As per the comments documentations for
ExistingUserController::PerformLoginFinishedActions(), this function performs
set of actions when login has been completed or has been cancelled. However in
this case it never gets called. 

Please let me know where the correct place it should be placed is, in order to
guarantee that it gets called.

[Nikita (slow), 2014/10/28 17:15:22]

I'm trying to verify why this call is not unblocking status area:

void ExistingUserController::OnProfilePrepared(Profile* profile,
                                               bool browser_launched) {
  // Reenable clicking on other windows and status area.
  login_display_->SetUIEnabled(true);
  ...
}

[afakhry, 2014/10/28 18:43:22]

Because LoginDisplayHostImpl::OnBrowserCreated() calls
LoginDisplayHostImpl::ResetLoginWindowAndView() (which resets 'login_view_' to
NULL) before the call to WebUILoginDisplay::SetUIEnabled() inside
ExistingUserController::OnProfilePrepared() uses the login_view (which is now
null) to unlock the status area.

 }
 
 void ExistingUserController::PerformLogin(
     const UserContext& user_context,
     LoginPerformer::AuthorizationMode auth_mode) {
   // TODO(antrim): remove this output once crash reason is found.
   LOG(ERROR) << "Setting flow from PerformLogin";
   ChromeUserManager::Get()
       ->GetUserFlow(user_context.GetUserID())
       ->SetHost(host_);
@@ skipping 737 matching lines @@
   is_login_in_progress_ = false;
 
   // Reenable clicking on other windows and status area.
   login_display_->SetUIEnabled(true);
 
   if (start_public_session_timer)
     StartPublicSessionAutoLoginTimer();
 }
 
 }  // namespace chromeos