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..901fffad4a0c4535a38a010cf781da7d97ef4951 100644
--- a/chrome/browser/extensions/external_policy_loader.cc
+++ b/chrome/browser/extensions/external_policy_loader.cc
@@ -4,14 +4,16 @@
 
 #include "chrome/browser/extensions/external_policy_loader.h"
 
+#include "base/logging.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/external_provider_impl.h"
 
 namespace extensions {
 
-ExternalPolicyLoader::ExternalPolicyLoader(ExtensionManagement *settings)
-    : settings_(settings) {
+ExternalPolicyLoader::ExternalPolicyLoader(ExtensionManagement* settings,
+                                           InstallationType type)
+    : settings_(settings), type_(type) {
   settings_->AddObserver(this);
 }
 
@@ -33,7 +35,16 @@ void ExternalPolicyLoader::AddExtension(base::DictionaryValue* dict,
 }
 
 void ExternalPolicyLoader::StartLoading() {
-  prefs_ = settings_->GetForceInstallList();
+  switch (type_) {
+    case FORCED:
+      prefs_ = settings_->GetForceInstallList();
+      break;
+    case RECOMMENDED:
+      prefs_ = settings_->GetRecommendedInstallList();
+      break;
+    default:
+      NOTREACHED();

[Finnur, 2014/10/06 19:35:27]

Leaving out the default case altogether is better than adding a default with a
NOTREACHED, because then the mistake (forgetting to add to this switch
statement) is caught at compile time, and not at runtime...

[binjin, 2014/10/07 09:29:23]

Done.

+  }
   LoadFinished();
 }