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

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

Issue 608283003: Remove retail mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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/existing_user_controller.cc
diff --git a/chrome/browser/chromeos/login/existing_user_controller.cc b/chrome/browser/chromeos/login/existing_user_controller.cc
index 9b9342215f060a0623c9108b4234c2632c68a40a..578d16d000fbcc8a7874f7b4f77312b19816c9a4 100644
--- a/chrome/browser/chromeos/login/existing_user_controller.cc
+++ b/chrome/browser/chromeos/login/existing_user_controller.cc
@@ -28,7 +28,6 @@
#include "chrome/browser/chromeos/boot_times_loader.h"
#include "chrome/browser/chromeos/customization_document.h"
#include "chrome/browser/chromeos/first_run/first_run.h"
-#include "chrome/browser/chromeos/kiosk_mode/kiosk_mode_settings.h"
#include "chrome/browser/chromeos/login/helper.h"
#include "chrome/browser/chromeos/login/login_utils.h"
#include "chrome/browser/chromeos/login/startup_utils.h"
@@ -414,10 +413,6 @@ void ExistingUserController::Login(const UserContext& user_context,
user_manager::USER_TYPE_PUBLIC_ACCOUNT) {
LoginAsPublicSession(user_context);
return;
- } else if (user_context.GetUserType() ==
- user_manager::USER_TYPE_RETAIL_MODE) {
- LoginAsRetailModeUser();
- return;
} else if (user_context.GetUserType() == user_manager::USER_TYPE_KIOSK_APP) {
LoginAsKioskApp(user_context.GetUserID(), specifics.kiosk_diagnostic_mode);
return;
@@ -473,24 +468,6 @@ void ExistingUserController::PerformLogin(
l10n_util::GetStringUTF8(IDS_CHROMEOS_ACC_LOGIN_SIGNING_IN));
}
-void ExistingUserController::LoginAsRetailModeUser() {
- // Stop the auto-login timer when attempting login.
- StopPublicSessionAutoLoginTimer();
-
- // Disable clicking on other windows.
- login_display_->SetUIEnabled(false);
- // TODO(rkc): Add a CHECK to make sure retail mode logins are allowed once
- // the enterprise policy wiring is done for retail mode.
-
- // Only one instance of LoginPerformer should exist at a time.
- login_performer_.reset(NULL);
- login_performer_.reset(new LoginPerformer(this));
- is_login_in_progress_ = true;
- login_performer_->LoginRetailMode();
- SendAccessibilityAlert(
- l10n_util::GetStringUTF8(IDS_CHROMEOS_ACC_LOGIN_SIGNIN_DEMOUSER));
-}
-
void ExistingUserController::LoginAsGuest() {
if (is_login_in_progress_ ||
user_manager::UserManager::Get()->IsUserLoggedIn()) {
@@ -1124,26 +1101,12 @@ gfx::NativeWindow ExistingUserController::GetNativeWindow() const {
void ExistingUserController::InitializeStartUrls() const {
std::vector<std::string> start_urls;
- const base::ListValue *urls;
user_manager::UserManager* user_manager = user_manager::UserManager::Get();
bool can_show_getstarted_guide =
user_manager->GetActiveUser()->GetType() ==
user_manager::USER_TYPE_REGULAR &&
!user_manager->IsCurrentUserNonCryptohomeDataEphemeral();
- if (user_manager->IsLoggedInAsDemoUser()) {
- if (CrosSettings::Get()->GetList(kStartUpUrls, &urls)) {
- // The retail mode user will get start URLs from a special policy if it is
- // set.
- for (base::ListValue::const_iterator it = urls->begin();
- it != urls->end(); ++it) {
- std::string url;
- if ((*it)->GetAsString(&url))
- start_urls.push_back(url);
- }
- }
- can_show_getstarted_guide = false;
- // Skip the default first-run behavior for public accounts.
Nikita (slow) 2014/10/01 07:13:01 nit: Please leave public session comment.
rkc 2014/10/01 17:29:52 Done.
- } else if (!user_manager->IsLoggedInAsPublicAccount()) {
+ if (!user_manager->IsLoggedInAsPublicAccount()) {
if (AccessibilityManager::Get()->IsSpokenFeedbackEnabled()) {
const char* url = kChromeVoxTutorialURLPattern;
PrefService* prefs = g_browser_process->local_state();

Powered by Google App Engine
This is Rietveld 408576698