Policy: Re-enabled plugin still disabled

chrome/browser/plugin_updater.h

Index: chrome/browser/plugin_updater.h
diff --git a/chrome/browser/plugin_updater.h b/chrome/browser/plugin_updater.h
index dc185954c2c7bdf0aee61fe94c2ab49dc88171ac..3bdd802fe9ae506f15d31878b5b0f84f6ad18af2 100644
--- a/chrome/browser/plugin_updater.h
+++ b/chrome/browser/plugin_updater.h
@@ -35,8 +35,15 @@ class PluginUpdater : public NotificationObserver {
   // Enable or disable a plugin group.
   void EnablePluginGroup(bool enable, const string16& group_name);
 
-  // Enable or disable a specific plugin file.
-  void EnablePluginFile(bool enable, const FilePath::StringType& file_path);
+  // Enable or disable a specific plugin file. If this function is called for
+  // setting the state of a plugin that has not been loaded yet (as is the case)

[Bernhard Bauer, 2010/12/21 19:07:39]

Nit: extend parenthesis to include "with policy controlled plugins"; doesn't
make much sense otherwise.

[pastarmovj, 2010/12/23 13:00:19]

Done.

+  // with policy controlled plugins then the |name| parameter must be set to the
+  // plugin name so that the right group for the plugin can be found/created.
+  // If this function is called for a loaded plugin (UI call) |name| can be left
+  // empty.
+  void EnablePlugin(bool enable,
+                    const FilePath::StringType& file_path,
+                    const string16& name);
 
   // Disable all plugin groups as defined by the user's preference file.
   void DisablePluginGroupsFromPrefs(Profile* profile);