new implementation of default apps

chrome/browser/extensions/external_provider_impl.cc

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 4ac241d1f1cdfd99b9dff364588b5815e928bd3f..fd4e52eab324563712150a1163642812b464f40f 100644
--- a/chrome/browser/extensions/external_provider_impl.cc
+++ b/chrome/browser/extensions/external_provider_impl.cc
@@ -394,6 +394,8 @@ void ExternalProviderImpl::CreateExternalProviders(
               Extension::NO_FLAGS)));
 
 #if !defined(OS_CHROMEOS)
+  // The default apps are installed as INTERNAL but use the external

[Mihai Parparita -not on Chrome, 2012/08/08 00:22:43]

This is clever, since it means that you don't have to reinvent a mechanism for
installing a list of extensions. However, I can see it being a bit confusing
(since it uses the "external" name).

Aaron, does this seem OK to you?

[Aaron Boodman, 2012/08/08 22:23:29]

It seems reasonable to me that providers can specify what location they want to
be installed as, and that they only participate in the rest of the protocol if
they pick one of external_*.

The naming is a bit confusing due to legacy... maybe these are really just
ExtensionProviders, or ExtensionInstallSources, or something. It would be nice
to rename it so that its behavior is better described. But that could be a
separate CL.

[Gaurav, 2012/08/10 23:06:55]

It may require more re-factoring and isolation. The kind at some places assumes
that all extensions provided are external and uninstalls them if they are not
provided anymore.
Will do in a followup CL.
On 2012/08/08 22:23:29, Aaron Boodman wrote:

+  // extension installer codeflow.
   provider_list->push_back(
       linked_ptr<ExternalProviderInterface>(
           new default_apps::Provider(
@@ -401,7 +403,7 @@ void ExternalProviderImpl::CreateExternalProviders(
               service,
               new ExternalPrefLoader(chrome::DIR_DEFAULT_APPS,
                                      ExternalPrefLoader::NONE),
-              Extension::EXTERNAL_PREF,
+              Extension::INTERNAL,
               Extension::INVALID,
               Extension::FROM_WEBSTORE)));
 #endif