Add recommended extension installation support

chrome/browser/extensions/external_policy_loader.cc

Index: chrome/browser/extensions/external_policy_loader.cc
diff --git a/chrome/browser/extensions/external_policy_loader.cc b/chrome/browser/extensions/external_policy_loader.cc
index 5297519142efebef28521165b7a2433fb17c1b10..97d5087b893e36959d83d456b34a55186ec1ee28 100644
--- a/chrome/browser/extensions/external_policy_loader.cc
+++ b/chrome/browser/extensions/external_policy_loader.cc
@@ -10,8 +10,9 @@
 
 namespace extensions {
 
-ExternalPolicyLoader::ExternalPolicyLoader(ExtensionManagement *settings)
-    : settings_(settings) {
+ExternalPolicyLoader::ExternalPolicyLoader(ExtensionManagement* settings,
+                                           InstallationType type)
+    : settings_(settings), type_(type) {
   settings_->AddObserver(this);
 }
 
@@ -33,7 +34,10 @@ void ExternalPolicyLoader::AddExtension(base::DictionaryValue* dict,
 }
 
 void ExternalPolicyLoader::StartLoading() {
-  prefs_ = settings_->GetForceInstallList();
+  if (type_ == FORCED)
+    prefs_ = settings_->GetForceInstallList();
+  else  // type_ == RECOMMENDED
+    prefs_ = settings_->GetRecommendedInstallList();

[Finnur, 2014/10/06 14:01:42]

I usually prefer a switch with no default: case for things like this. Then the
person who adds a new value to the InstallationType enum gets a compile error
here and realizes the need to update this function.

[binjin, 2014/10/06 18:48:47]

Done.

   LoadFinished();
 }