Commit bbe95656 authored by tbarzic@chromium.org's avatar tbarzic@chromium.org

Mark default apps on ChromeOS installed from DIR_EXTERNAL_EXTENSIONS "from webstore"

The default extensions should be able to load NaCl modules.
This is a problem now because they aren't really distinguishable from normal extensions. To fix
this, mark them as "from webstore" when they're loaded.

Also, revert crrev.com/130235 as that didn't work

BUG=chromium-os:28707
TEST=none

Review URL: https://chromiumcodereview.appspot.com/10269031

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134761 0039d316-1c4b-4281-b951-d872f2087c98
parent 23689f17
...@@ -315,25 +315,15 @@ void ExternalExtensionProviderImpl::CreateExternalProviders( ...@@ -315,25 +315,15 @@ void ExternalExtensionProviderImpl::CreateExternalProviders(
check_admin_permissions_on_mac), check_admin_permissions_on_mac),
Extension::EXTERNAL_PREF, Extension::EXTERNAL_PREF,
Extension::EXTERNAL_PREF_DOWNLOAD, Extension::EXTERNAL_PREF_DOWNLOAD,
Extension::NO_FLAGS)));
#if defined(OS_CHROMEOS) #if defined(OS_CHROMEOS)
// Define a per-user source of external default extensions, which serves
// as a source for OEM customization. Mark these default extensions as
// being from the webstore so they can load Native Client modules.
provider_list->push_back(
linked_ptr<ExternalExtensionProviderInterface>(
new ExternalExtensionProviderImpl(
service,
new ExternalPrefExtensionLoader(
chrome::DIR_USER_EXTERNAL_EXTENSIONS,
ExternalPrefExtensionLoader::NONE),
Extension::EXTERNAL_PREF,
Extension::EXTERNAL_PREF_DOWNLOAD,
Extension::FROM_WEBSTORE))); Extension::FROM_WEBSTORE)));
#else
Extension::NO_FLAGS)));
#endif #endif
#if defined (OS_MACOSX)
#if defined(OS_CHROMEOS) || defined (OS_MACOSX)
// Define a per-user source of external extensions. // Define a per-user source of external extensions.
// On Chrome OS, this serves as a source for OEM customization.
provider_list->push_back( provider_list->push_back(
linked_ptr<ExternalExtensionProviderInterface>( linked_ptr<ExternalExtensionProviderInterface>(
new ExternalExtensionProviderImpl( new ExternalExtensionProviderImpl(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment