Implement kiosk multiple apps feature.

chrome/browser/extensions/extension_service.cc

Index: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 96e5fbff576f359e088cb9e4f2ed6823668c5b42..d89574777ed7917c328198769996f142fb535699 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -202,7 +202,12 @@ bool ExtensionService::OnExternalExtensionUpdateUrlFound(
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   CHECK(crx_file::id_util::IdIsValid(id));
 
-  if (Manifest::IsExternalLocation(location)) {
+  // External extensions can be cached because they're the
+  // same across users, but it's necessary to exclude kiosk apps because
+  // the kiosk apps should be constrained in the isolated kiosk session
+  // and should not be shared across.

[not at google - send to devlin, 2015/08/31 18:11:52]

This comment isn't a complete sentence.

[jennyz, 2015/08/31 23:57:25]

Done.

+  if (Manifest::IsExternalLocation(location) &&
+      !extensions::ExtensionsBrowserClient::Get()->IsRunningInForcedAppMode()) {

[not at google - send to devlin, 2015/08/31 18:11:52]

Without this change, does the test you added fail?

[jennyz, 2015/08/31 23:57:25]

Yes, the test will fail since the cache write to a directory under /var which is
not accessible under test environment.

[jennyz, 2015/09/01 16:53:09]

Actually the tests passed now, since I have moved the hard coded cache path to
chromeos PathProvider. So I removed the special case code for kiosk mode here.

     // All extensions that are not user specific can be cached.
     extensions::ExtensionsBrowserClient::Get()->GetExtensionCache()
         ->AllowCaching(id);