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 14 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);
+  void EnablePlugin(bool enable,
+                    const FilePath::StringType& file_path,
+                    const string16& name);

[jam, 2010/12/17 19:14:45]

can you add documentation to what "name" is?  why is it needed now?

[pastarmovj, 2010/12/20 19:57:37]

Done.

[Bernhard Bauer, 2010/12/20 22:30:28]

Umm...?

 
   // 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 34 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_