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

Unified Diff: chrome/browser/ui/webui/extensions/chromeos/kiosk_apps_handler.cc

Issue 130613004: Enabled consumer kiosk mode for all. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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/ui/webui/extensions/chromeos/kiosk_apps_handler.cc
diff --git a/chrome/browser/ui/webui/extensions/chromeos/kiosk_apps_handler.cc b/chrome/browser/ui/webui/extensions/chromeos/kiosk_apps_handler.cc
index 3fdaecf29bafcd48407a226b62f26d080cd733d3..e7cd9538d7857486e4b8b39da46de952cf9365bf 100644
--- a/chrome/browser/ui/webui/extensions/chromeos/kiosk_apps_handler.cc
+++ b/chrome/browser/ui/webui/extensions/chromeos/kiosk_apps_handler.cc
@@ -94,6 +94,7 @@ KioskAppsHandler::KioskAppsHandler()
: kiosk_app_manager_(KioskAppManager::Get()),
initialized_(false),
is_kiosk_enabled_(false),
+ is_auto_launch_enabled_(false),
weak_ptr_factory_(this) {
kiosk_app_manager_->AddObserver(this);
}
@@ -195,18 +196,23 @@ void KioskAppsHandler::OnKioskAppDataLoadFailure(const std::string& app_id) {
}
-void KioskAppsHandler::OnGetConsumerKioskModeStatus(
- chromeos::KioskAppManager::ConsumerKioskModeStatus status) {
+void KioskAppsHandler::OnGetConsumerKioskAutoLaunchStatus(
+ chromeos::KioskAppManager::ConsumerKioskAutoLaunchStatus status) {
initialized_ = true;
is_kiosk_enabled_ =
- ((status == KioskAppManager::CONSUMER_KIOSK_MODE_ENABLED) &&
- chromeos::UserManager::Get()->IsCurrentUserOwner()) ||
+ chromeos::UserManager::Get()->IsCurrentUserOwner() ||
+ !base::SysInfo::IsRunningOnChromeOS();
+
+ is_auto_launch_enabled_ =
+ status == KioskAppManager::CONSUMER_KIOSK_AUTO_LAUNCH_ENABLED ||
!base::SysInfo::IsRunningOnChromeOS();
if (is_kiosk_enabled_) {
- base::FundamentalValue enabled(is_kiosk_enabled_);
+ base::DictionaryValue kiosk_params;
+ kiosk_params.SetBoolean("kioskEnabled", is_kiosk_enabled_);
+ kiosk_params.SetBoolean("autoLaunchEnabled", is_auto_launch_enabled_);
web_ui()->CallJavascriptFunction("extensions.KioskAppsOverlay.enableKiosk",
- enabled);
+ kiosk_params);
}
}
@@ -248,8 +254,8 @@ void KioskAppsHandler::SendKioskAppSettings() {
void KioskAppsHandler::HandleInitializeKioskAppSettings(
const base::ListValue* args) {
- KioskAppManager::Get()->GetConsumerKioskModeStatus(
- base::Bind(&KioskAppsHandler::OnGetConsumerKioskModeStatus,
+ KioskAppManager::Get()->GetConsumerKioskAutoLaunchStatus(
+ base::Bind(&KioskAppsHandler::OnGetConsumerKioskAutoLaunchStatus,
weak_ptr_factory_.GetWeakPtr()));
}
@@ -286,7 +292,7 @@ void KioskAppsHandler::HandleRemoveKioskApp(const base::ListValue* args) {
void KioskAppsHandler::HandleEnableKioskAutoLaunch(
const base::ListValue* args) {
- if (!initialized_ || !is_kiosk_enabled_)
+ if (!initialized_ || !is_kiosk_enabled_ || !is_auto_launch_enabled_)
return;
std::string app_id;
@@ -297,7 +303,7 @@ void KioskAppsHandler::HandleEnableKioskAutoLaunch(
void KioskAppsHandler::HandleDisableKioskAutoLaunch(
const base::ListValue* args) {
- if (!initialized_ || !is_kiosk_enabled_)
+ if (!initialized_ || !is_kiosk_enabled_ || !is_auto_launch_enabled_)
return;
std::string app_id;

Powered by Google App Engine
This is Rietveld 408576698