Add recommended extension installation support

chrome/browser/extensions/extension_management.h

Index: chrome/browser/extensions/extension_management.h
diff --git a/chrome/browser/extensions/extension_management.h b/chrome/browser/extensions/extension_management.h
index c3eccda371e95de69224f680804df7892612f257..adface9fdabc53276579aa3e1d590ff8b93cea7d 100644
--- a/chrome/browser/extensions/extension_management.h
+++ b/chrome/browser/extensions/extension_management.h
@@ -143,9 +143,13 @@ class ExtensionManagement : public KeyedService {
   // from the command line, or when loaded as an unpacked extension).
   bool BlacklistedByDefault();
 
-  // Returns the force install list, in format specified by
-  // ExternalPolicyLoader::AddExtension().
-  scoped_ptr<base::DictionaryValue> GetForceInstallList() const;
+  // Returns a list of extensions which will be automatically installed, in
+  // format specified in ExternalPolicyLoader::AddExtension().
+  // If |forced| is true, returned extensions are not permitted to be
+  // disabled by user, otherwise returned extensions are allowed to be
+  // disabled but not uninstalled.
+  // Note that these two lists have no extension in common.
+  scoped_ptr<base::DictionaryValue> GetAutoInstallList(bool forced) const;

[Joao da Silva, 2014/09/18 12:21:23]

Use two methods instead of having the argument:

scoped_ptr<base::DictionaryValue> GetForceInstallList() const;

scoped_ptr<base::DictionaryValue> GetRecommendedInstallList() const;

[binjin, 2014/09/18 16:25:15]

Done. I choosed to duplicated code here. since adding helper function to private
section will complicated interface again.

 
   // Returns if an extension with id |id| is allowed to install or not.
   bool IsInstallationAllowed(const ExtensionId& id) const;