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

Unified Diff: chrome/browser/extensions/external_provider_impl.cc

Issue 1301323005: Implement kiosk multiple apps feature. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 5 years, 4 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/extensions/external_provider_impl.cc
diff --git a/chrome/browser/extensions/external_provider_impl.cc b/chrome/browser/extensions/external_provider_impl.cc
index 7844bca135e6f2f5010e100d8bf1aab0e5fec90c..9078c1da7299b30f450205ea799a4b0089c4fe50 100644
--- a/chrome/browser/extensions/external_provider_impl.cc
+++ b/chrome/browser/extensions/external_provider_impl.cc
@@ -492,6 +492,8 @@ void ExternalProviderImpl::CreateExternalProviders(
chromeos::KioskAppManager* kiosk_app_manager =
chromeos::KioskAppManager::Get();
DCHECK(kiosk_app_manager);
not at google - send to devlin 2015/08/26 17:53:58 When will this be null? It's a pain to DCHECK it h
jennyz 2015/08/28 18:24:08 Done.
+
+ // Kiosk primary app external provider.
if (kiosk_app_manager && !kiosk_app_manager->external_loader_created()) {
scoped_ptr<ExternalProviderImpl> kiosk_app_provider(
new ExternalProviderImpl(
@@ -503,6 +505,20 @@ void ExternalProviderImpl::CreateExternalProviders(
provider_list->push_back(
linked_ptr<ExternalProviderInterface>(kiosk_app_provider.release()));
}
+
+ // Kiosk secondary app external provider.
+ if (kiosk_app_manager &&
+ !kiosk_app_manager->secondary_app_external_loader_created()) {
+ scoped_ptr<ExternalProviderImpl> secondary_kiosk_app_provider(
+ new ExternalProviderImpl(
+ service, kiosk_app_manager->CreateSecondaryAppExternalLoader(),
+ profile, Manifest::EXTERNAL_PREF,
+ Manifest::EXTERNAL_PREF_DOWNLOAD, Extension::NO_FLAGS));
+ secondary_kiosk_app_provider->set_auto_acknowledge(true);
+ secondary_kiosk_app_provider->set_install_immediately(true);
+ provider_list->push_back(linked_ptr<ExternalProviderInterface>(
+ secondary_kiosk_app_provider.release()));
+ }
#endif
return;
}

Powered by Google App Engine
This is Rietveld 408576698