Add recommended extension installation support

chrome/browser/extensions/external_policy_loader.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/external_policy_loader.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);
 }
 
 ExternalPolicyLoader::~ExternalPolicyLoader() {
   settings_->RemoveObserver(this);
 }
 
 void ExternalPolicyLoader::OnExtensionManagementSettingsChanged() {
   StartLoading();
 }
 
 // static
 void ExternalPolicyLoader::AddExtension(base::DictionaryValue* dict,
                                         const std::string& extension_id,
                                         const std::string& update_url) {
   dict->SetString(base::StringPrintf("%s.%s", extension_id.c_str(),
                                      ExternalProviderImpl::kExternalUpdateUrl),
                   update_url);
 }
 
 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();
 }
 
 }  // namespace extensions