Simplify LoginState methods.

chrome/browser/chromeos/options/wifi_config_view.cc

Index: chrome/browser/chromeos/options/wifi_config_view.cc
diff --git a/chrome/browser/chromeos/options/wifi_config_view.cc b/chrome/browser/chromeos/options/wifi_config_view.cc
index b2d464c492c0cf43be955e88489ffe268bb4a657..8a6f9304b258f88ce7da32b931e0e875d2d080d1 100644
--- a/chrome/browser/chromeos/options/wifi_config_view.cc
+++ b/chrome/browser/chromeos/options/wifi_config_view.cc
@@ -575,7 +575,7 @@ void WifiConfigView::UpdateErrorLabel() {
   if (UserCertRequired() && CertLibrary::Get()->CertificatesLoaded()) {
     if (!HaveUserCerts()) {
       if (!LoginState::Get()->IsUserLoggedIn() ||
-          LoginState::Get()->IsGuestUser()) {
+          LoginState::Get()->IsGuestSessionUser()) {

[stevenjb, 2014/08/08 16:57:06]

Do installed certificates in public sessions persist? I guess if it is a public
session then 'login' isn't an option anyway? Maybe we should add a comment to
that effect.

[pneubeck (no reviews), 2014/08/08 20:45:24]

Yes, right. I don't want to change behavior in this CL so I added the public
session to this condition.

In the follow up, I will remove public session from this condition as the TPM is
loaded and policy can install certs. I think it would be irritating to tell the
user to login in. The other message seems to be more helpful.

[stevenjb, 2014/08/12 02:23:17]

sgtm

         error_msg = l10n_util::GetStringUTF16(
             IDS_OPTIONS_SETTINGS_INTERNET_OPTIONS_LOGIN_FOR_USER_CERT);
       } else {