Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3540)

Unified Diff: chrome/browser/chromeos/power/power_button_observer.cc

Issue 13495003: Add LoginState class to src/chromeos/login (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/power/power_button_observer.cc
diff --git a/chrome/browser/chromeos/power/power_button_observer.cc b/chrome/browser/chromeos/power/power_button_observer.cc
index 793f00ea88acb3ac96b2214f00a6bf1230dd36b3..3205dcb1227fc22e69ddf5470cc825c5e1fb4683 100644
--- a/chrome/browser/chromeos/power/power_button_observer.cc
+++ b/chrome/browser/chromeos/power/power_button_observer.cc
@@ -5,12 +5,11 @@
#include "chrome/browser/chromeos/power/power_button_observer.h"
#include "ash/shell.h"
+#include "ash/system/tray/system_tray_delegate.h"
#include "ash/system/user/login_status.h"
#include "ash/wm/power_button_controller.h"
#include "base/logging.h"
#include "chrome/browser/chromeos/login/screen_locker.h"
-#include "chrome/browser/chromeos/login/user.h"
-#include "chrome/browser/chromeos/login/user_manager.h"
#include "chrome/browser/chromeos/power/session_state_controller_delegate_chromeos.h"
#include "chrome/common/chrome_notification_types.h"
#include "chromeos/dbus/dbus_thread_manager.h"
@@ -21,29 +20,10 @@ namespace chromeos {
namespace {
ash::user::LoginStatus GetCurrentLoginStatus() {
- const UserManager* user_manager = UserManager::Get();
- if (!user_manager->IsUserLoggedIn())
- return ash::user::LOGGED_IN_NONE;
- if (user_manager->IsCurrentUserOwner())
- return ash::user::LOGGED_IN_OWNER;
-
- switch (user_manager->GetLoggedInUser()->GetType()) {
- case User::USER_TYPE_REGULAR:
- return ash::user::LOGGED_IN_USER;
- case User::USER_TYPE_GUEST:
- return ash::user::LOGGED_IN_GUEST;
- case User::USER_TYPE_RETAIL_MODE:
- return ash::user::LOGGED_IN_RETAIL_MODE;
- case User::USER_TYPE_PUBLIC_ACCOUNT:
- return ash::user::LOGGED_IN_PUBLIC;
- case User::USER_TYPE_LOCALLY_MANAGED:
- return ash::user::LOGGED_IN_LOCALLY_MANAGED;
- case User::USER_TYPE_KIOSK_APP:
- return ash::user::LOGGED_IN_KIOSK_APP;
- }
-
- NOTREACHED();
- return ash::user::LOGGED_IN_USER;
+ if (!ash::Shell::GetInstance()->system_tray_delegate())
+ return ash::user::LOGGED_IN_NONE;
bartfab (slow) 2013/04/08 15:18:36 The double space after return is still there.
stevenjb 2013/04/09 15:12:26 Done.
+ return ash::Shell::GetInstance()->system_tray_delegate()->
+ GetUserLoginStatus();
}
} // namespace
« no previous file with comments | « chrome/browser/chromeos/options/wimax_config_view.cc ('k') | chrome/browser/chromeos/status/data_promo_notification.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698