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

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

The network configuration dialog for secure networks is not displaying
correctly during the kiosk splash screen, because a kiosk user is logged in.
Update LoginState to track the session started status.

BUG=269182
TEST=manually on device

[Tim Song, 2013-09-13 02:15:52]

This patch adds session awareness to LoginState and use it to determine the
ash::user::LoginStatus. Please take a look and let me know if this approach
works.

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

https://codereview.chromium.org/24072006/diff/9001/chrome/browser/chromeos/lo...
File chrome/browser/chromeos/login/user_manager_impl.cc (right):

https://codereview.chromium.org/24072006/diff/9001/chrome/browser/chromeos/lo...
chrome/browser/chromeos/login/user_manager_impl.cc:1731:
LoginState::Get()->SetSessionStarted(session_started_);
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...

[Tim Song, 2013-09-13 18:52:49]

Hmm... it looks like we can already distinguish between "logged in" and "session
started" with SessionStateDelegate::IsActiveUserSessionStarted(), so I agree we
shouldn't add additional complexity to LoginState.

I'll close this review and go with another approach. Thank you for the review!