Policy: Re-enabled plugin still disabled

chrome/browser/plugin_updater.h

 // Copyright (c) 2010 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_PLUGIN_UPDATER_H_
 #define CHROME_BROWSER_PLUGIN_UPDATER_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 17 matching lines @@
 
 class PluginUpdater : public NotificationObserver {
  public:
   // Get a list of all the plugin groups. The caller should take ownership
   // of the returned ListValue.
   static ListValue* GetPluginGroupsData();
 
   // 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);
 
   // Write the enable/disable status to the user's preference file.
   void UpdatePreferences(Profile* profile, int delay_ms);
 
   // NotificationObserver method overrides
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
@@ skipping 35 matching lines @@
   // Note: if you change this to false from true, you must update
   // kPluginsEnabledInternalPDF to be a new name (i.e. add 2, 3, 4...) at end.
   bool enable_internal_pdf_;
 
   bool notify_pending_;
 
   DISALLOW_COPY_AND_ASSIGN(PluginUpdater);
 };
 
 #endif  // CHROME_BROWSER_PLUGIN_UPDATER_H_