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

Unified Diff: chrome/browser/chromeos/login/user_manager_impl.cc

Issue 10590004: [cros] Fake a stub user login when no --login-manager and --login-user are given. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 8 years, 5 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/login/user_manager_impl.cc
diff --git a/chrome/browser/chromeos/login/user_manager_impl.cc b/chrome/browser/chromeos/login/user_manager_impl.cc
index cfd1f3ec19fd4989840439b4816a6f62777286bb..80e5b96bb8d8ced388d377d1558934904ba78b9d 100644
--- a/chrome/browser/chromeos/login/user_manager_impl.cc
+++ b/chrome/browser/chromeos/login/user_manager_impl.cc
@@ -70,9 +70,6 @@ namespace {
// depends on that and it's hard to figure out what).
const char kGuestUser[] = "";
-// Stub user email (for test paths).
-const char kStubUser[] = "stub-user@example.com";
-
// Names of nodes with info about user image.
const char kImagePathNodeName[] = "path";
const char kImageIndexNodeName[] = "index";
@@ -95,9 +92,6 @@ const int kDefaultOOBEWallpaperIndex = 0; // IDR_AURA_WALLPAPERS_ROMAINGUY_0
const int kThumbnailWidth = 128;
const int kThumbnailHeight = 80;
-// Index of the default image used for the |kStubUser| user.
-const int kStubDefaultImageIndex = 0;
-
// Delay betweeen user login and attempt to update user's profile data.
const long kProfileDataDownloadDelayMs = 10000;
@@ -232,18 +226,8 @@ UserManagerImpl::UserManagerImpl()
downloading_profile_image_(false) {
// UserManager instance should be used only on UI thread.
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- // If we're not running on ChromeOS, and are not showing the login manager
- // or attempting a command line login? Then login the stub user.
- CommandLine* command_line = CommandLine::ForCurrentProcess();
- if (!base::chromeos::IsRunningOnChromeOS() &&
- !command_line->HasSwitch(switches::kLoginManager) &&
- !command_line->HasSwitch(switches::kLoginPassword) &&
- !command_line->HasSwitch(switches::kGuestSession)) {
- StubUserLoggedIn();
- }
MigrateWallpaperData();
-
registrar_.Add(this, chrome::NOTIFICATION_OWNER_KEY_FETCH_ATTEMPT_SUCCEEDED,
content::NotificationService::AllSources());
registrar_.Add(this, chrome::NOTIFICATION_PROFILE_ADDED,
@@ -269,14 +253,6 @@ void UserManagerImpl::UserLoggedIn(const std::string& email,
bool browser_restart) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- // Remove the stub user if it is still around.
- if (logged_in_user_) {
- DCHECK(IsLoggedInAsStub());
- delete logged_in_user_;
- logged_in_user_ = NULL;
- is_current_user_ephemeral_ = false;
- }
-
if (email == kGuestUser) {
GuestUserLoggedIn();
return;
@@ -401,14 +377,6 @@ void UserManagerImpl::EphemeralUserLoggedIn(const std::string& email) {
NotifyOnLogin();
}
-void UserManagerImpl::StubUserLoggedIn() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- is_current_user_ephemeral_ = true;
- logged_in_user_ = new User(kStubUser, false);
- logged_in_user_->SetImage(UserImage(GetDefaultImage(kStubDefaultImageIndex)),
- kStubDefaultImageIndex);
-}
-
void UserManagerImpl::InitializeWallpaper() {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
if (!IsUserLoggedIn()) {
@@ -431,10 +399,6 @@ void UserManagerImpl::InitializeWallpaper() {
}
}
return;
- } else if (IsLoggedInAsStub()) {
- ash::Shell::GetInstance()->desktop_background_controller()->
- SetDefaultWallpaper(ash::GetInvalidWallpaperIndex());
- return;
}
UserSelected(GetLoggedInUser().email());
}
@@ -722,7 +686,7 @@ void UserManagerImpl::Observe(int type,
base::Unretained(this)));
break;
case chrome::NOTIFICATION_PROFILE_ADDED:
- if (IsUserLoggedIn() && !IsLoggedInAsGuest() && !IsLoggedInAsStub()) {
+ if (IsUserLoggedIn() && !IsLoggedInAsGuest()) {
Profile* profile = content::Source<Profile>(source).ptr();
if (!profile->IsOffTheRecord() &&
profile == ProfileManager::GetDefaultProfile()) {
@@ -740,7 +704,7 @@ void UserManagerImpl::Observe(int type,
}
void UserManagerImpl::OnStateChanged() {
- DCHECK(IsUserLoggedIn() && !IsLoggedInAsGuest() && !IsLoggedInAsStub());
+ DCHECK(IsUserLoggedIn() && !IsLoggedInAsGuest());
AuthError::State state = observed_sync_service_->GetAuthError().state();
if (state != AuthError::NONE &&
state != AuthError::CONNECTION_FAILED &&
@@ -1005,8 +969,8 @@ bool UserManagerImpl::AreEphemeralUsersEnabled() const {
}
bool UserManagerImpl::IsEphemeralUser(const std::string& email) const {
- // The guest user always is ephemeral.
- if (email == kGuestUser)
+ // The guest and stub user always are ephemeral.
+ if (email == kGuestUser || email == kStubUser)
return true;
// The currently logged-in user is ephemeral iff logged in as ephemeral.
« no previous file with comments | « chrome/browser/chromeos/login/user_manager_impl.h ('k') | chrome/browser/chromeos/login/user_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698