Add recommended extension installation support

chrome/browser/extensions/extension_management.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_MANAGEMENT_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_MANAGEMENT_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 125 matching lines @@
 
   // Get the ManagementPolicy::Provider controlled by extension management
   // policy settings.
   ManagementPolicy::Provider* GetProvider();
 
   // Checks if extensions are blacklisted by default, by policy. When true,
   // this means that even extensions without an ID should be blacklisted (e.g.
   // 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;
 
   // Returns true if an extension download should be allowed to proceed.
   bool IsOffstoreInstallAllowed(const GURL& url, const GURL& referrer_url);
 
   // Helper function to read |settings_by_id_| with |id| as key. Returns a
   // constant reference to default settings if |id| does not exist.
   const IndividualSettings& ReadById(const ExtensionId& id) const;
@@ skipping 66 matching lines @@
       content::BrowserContext* context) const OVERRIDE;
   virtual content::BrowserContext* GetBrowserContextToUse(
       content::BrowserContext* context) const OVERRIDE;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionManagementFactory);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_MANAGEMENT_H_