Add LoginState class to src/chromeos/login

chrome/browser/google_apis/auth_service.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/google_apis/auth_service.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/message_loop_proxy.h"
 #include "base/metrics/histogram.h"
 #include "chrome/browser/google_apis/auth_service_observer.h"
 #include "chrome/browser/google_apis/operation_registry.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/token_service.h"
 #include "chrome/browser/signin/token_service_factory.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/notification_types.h"
 #include "google_apis/gaia/gaia_constants.h"
 #include "google_apis/gaia/gaia_urls.h"
 #include "google_apis/gaia/google_service_auth_error.h"
 #include "google_apis/gaia/oauth2_access_token_fetcher.h"
 
 #if defined(OS_CHROMEOS)
-#include "chrome/browser/chromeos/login/user_manager.h"
+#include "chromeos/login/login_state.h"
 #endif  // OS_CHROMEOS
 
 using content::BrowserThread;
 
 namespace google_apis {
 
 namespace {
 
 // Used for success ratio histograms. 0 for failure, 1 for success,
 // 2 for no connection (likely offline).
@@ skipping 243 matching lines @@
     refresh_token_.clear();
   }
   FOR_EACH_OBSERVER(AuthServiceObserver,
                     observers_,
                     OnOAuth2RefreshTokenChanged());
 }
 
 // static
 bool AuthService::CanAuthenticate(Profile* profile) {
 #if defined(OS_CHROMEOS)
-  if (!chromeos::UserManager::Get()->IsUserLoggedIn() ||
-      chromeos::UserManager::Get()->IsLoggedInAsGuest() ||
-      chromeos::UserManager::Get()->IsLoggedInAsDemoUser())
+  if (!chromeos::LoginState::IsInitialized())
     return false;
+  chromeos::LoginState::LoggedInUserType user_type =
+      chromeos::LoginState::Get()->GetLoggedInUserType();
+  if (user_type == chromeos::LoginState::LOGGED_IN_USER_NONE ||
+      user_type == chromeos::LoginState::LOGGED_IN_USER_GUEST ||
+      user_type == chromeos::LoginState::LOGGED_IN_USER_RETAIL_MODE) {

[bartfab (slow), 2013/04/10 18:51:20]

Can you add a TODO(bartfab): Check whether public session users should be
excluded here as well?

I cannot quite figure it out by looking at this snippet, but I think it is
possible we want to filter out public session users as well.

[stevenjb, 2013/04/10 20:46:41]

Yeah, I thought about that, and you're right, we should fix it here or add a
TODO. I'll ping Zel and fix it or add a comment.

+    return false;
+  }
 #endif  // OS_CHROMEOS
 
   // Authentication cannot be done with the incognito mode profile.
   if (profile->IsOffTheRecord())
     return false;
 
   return true;
 }
 
 }  // namespace google_apis