Add LoginState class to src/chromeos/login

chrome/browser/chromeos/background/ash_user_wallpaper_delegate.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/chromeos/background/ash_user_wallpaper_delegate.h"
 
 #include "ash/shell.h"
 #include "ash/desktop_background/user_wallpaper_delegate.h"
 #include "ash/wm/window_animations.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "chrome/browser/chromeos/extensions/wallpaper_manager_util.h"
-#include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/login/wallpaper_manager.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chromeos/chromeos_switches.h"
+#include "chromeos/login/login_state.h"
 #include "content/public/browser/notification_service.h"
 
 namespace chromeos {
 
 namespace {
 
 bool IsNormalWallpaperChange() {
-  if (chromeos::UserManager::Get()->IsUserLoggedIn() ||
+  if (chromeos::LoginState::Get()->IsUserLoggedIn() ||
       !CommandLine::ForCurrentProcess()->HasSwitch(switches::kFirstBoot) ||
       WizardController::IsZeroDelayEnabled() ||
       !CommandLine::ForCurrentProcess()->HasSwitch(switches::kLoginManager)) {
     return true;
   }
 
   return false;
 }
 
 class UserWallpaperDelegate : public ash::UserWallpaperDelegate {
@@ skipping 36 matching lines @@
 
   virtual void InitializeWallpaper() OVERRIDE {
     chromeos::WallpaperManager::Get()->InitializeWallpaper();
   }
 
   virtual void OpenSetWallpaperPage() OVERRIDE {
     wallpaper_manager_util::OpenWallpaperManager();
   }
 
   virtual bool CanOpenSetWallpaperPage() OVERRIDE {
-    return !chromeos::UserManager::Get()->IsLoggedInAsGuest();
+    LoginState::LoggedInUserType user_type =
+        LoginState::Get()->GetLoggedInUserType();
+    if (user_type == LoginState::LOGGED_IN_USER_REGULAR ||
+        user_type == LoginState::LOGGED_IN_USER_OWNER ||
+        user_type == LoginState::LOGGED_IN_USER_LOCALLY_MANAGED)
+      return true;

[Nikita (slow), 2013/04/10 08:09:22]

nit: add {} as condition takes more than one line.

[stevenjb, 2013/04/10 16:32:17]

Heh. I've been going back and forth with different reviewers / reviewees
regarding whether or not {} is required with a multi-line if() clause and single
line statement. There doesn't seem to be a hard rule, but I do tend to prefer
{}, so done.

+    return false;
   }
 
   virtual void OnWallpaperAnimationFinished() OVERRIDE {
     content::NotificationService::current()->Notify(
         chrome::NOTIFICATION_WALLPAPER_ANIMATION_FINISHED,
         content::NotificationService::AllSources(),
         content::NotificationService::NoDetails());
   }
 
   virtual void OnWallpaperBootAnimationFinished() OVERRIDE {
     // Make sure that boot animation type is used only once.
     boot_animation_finished_ = true;
   }
 
  private:
   bool boot_animation_finished_;
 
   DISALLOW_COPY_AND_ASSIGN(UserWallpaperDelegate);
 };
 
 }  // namespace
 
 ash::UserWallpaperDelegate* CreateUserWallpaperDelegate() {
   return new chromeos::UserWallpaperDelegate();
 }
 
 }  // namespace chromeos