Return correct window when user is logged in but before session is started.

chrome/browser/chromeos/login/user_manager_impl.cc

Index: chrome/browser/chromeos/login/user_manager_impl.cc
diff --git a/chrome/browser/chromeos/login/user_manager_impl.cc b/chrome/browser/chromeos/login/user_manager_impl.cc
index 8a2847601dbea3702b18724b3c9d73c72b65832c..0e22e34e5fbaf42c85e73dc80153e25568901258 100644
--- a/chrome/browser/chromeos/login/user_manager_impl.cc
+++ b/chrome/browser/chromeos/login/user_manager_impl.cc
@@ -1728,6 +1728,7 @@ void UserManagerImpl::UpdateLoginState() {
     login_user_type = LoginState::LOGGED_IN_USER_REGULAR;
 
   LoginState::Get()->SetLoggedInState(logged_in_state, login_user_type);
+  LoginState::Get()->SetSessionStarted(session_started_);

[stevenjb, 2013/09/13 18:22:34]

This is confusing to me. Do we really need to introduce a separate
'SessionStarted' property to LoginState? Couldn't we just set logged_in_state =
LOGGED_IN_NONE if session_started_ is false?

If there is code that uses LoginState that needs to differentiate between
LOGGED_IN_ACTIVE and LOGGED_IN_SESSION_STARTED, we should add that state instead
of a separate accessor. The intention was to keep LoginState relatively
simple...

 }
 
 void UserManagerImpl::SetLRUUser(User* user) {