Policy: Re-enabled plugin still disabled

webkit/glue/plugins/webplugininfo.h

 // Copyright (c) 2006-2008 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 WEBKIT_GLUE_WEBPLUGININFO_H_
-#define WEBKIT_GLUE_WEBPLUGININFO_H_
+#ifndef WEBKIT_GLUE_PLUGINS_WEBPLUGININFO_H_
+#define WEBKIT_GLUE_PLUGINS_WEBPLUGININFO_H_
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
 
 // Describes a mime type entry for a plugin.
 struct WebPluginMimeType {
   WebPluginMimeType();
   ~WebPluginMimeType();
 
   // The name of the mime type (e.g., "application/x-shockwave-flash").
   std::string mime_type;
 
   // A list of all the file extensions for this mime type.
   std::vector<std::string> file_extensions;
 
   // Description of the mime type.
   string16 description;
 };
 
 // Describes an available NPAPI plugin.
 struct WebPluginInfo {
+  enum Reason { USER = 1 << 1, MANAGED = 1 << 2 };
+
   WebPluginInfo();
   WebPluginInfo(const WebPluginInfo& rhs);
   ~WebPluginInfo();
   WebPluginInfo& operator=(const WebPluginInfo& rhs);
 
   // Special constructor only used during unit testing:
   WebPluginInfo(const string16& fake_name,
+                const FilePath& fake_path,
                 const string16& fake_version,
                 const string16& fake_desc);
 
+
+  // Enables the plugin if not already enabled and if policy allows it to.
+  // Returns true on success.
+  bool Enable(Reason reason);
+
+  // Disables the plugin if not already disabled and if policy allows it to.
+  // Returns true on success.
+  bool Disable(Reason reason);
+
+  bool IsEnabled() const { return enabled; }
+  static bool IsManaged(int reason) { return (reason & MANAGED) != 0; }
+  bool HasVersion() const { return version.length() != 0; }
+
+  // Returns true if the plugin supports |mime-type|. |mime_type| should be all

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

Mini-nit: |mime_type| with an underscore.

[pastarmovj, 2010/12/21 16:53:02]

Done.

+  // lower case.
+  bool SupportsType(const std::string& mime_type, bool allow_wildcard) const;
+
+  // Returns true if the given plugin supports a given file extension.
+  // |extension| should be all lower case. If |mime_type| is not NULL, it will
+  // be set to the MIME type if found. The mime type which corresponds to the

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

And please for the second instance as well :)

[pastarmovj, 2010/12/21 16:53:02]

Done.

+  // extension is optionally returned back.
+  bool SupportsExtension(const std::string& extension,
+                         std::string* actual_mime_type) const;
+
   // The name of the plugin (i.e. Flash).
   string16 name;
 
   // The path to the plugin file (DLL/bundle/library).
   FilePath path;
 
   // The version number of the plugin file (may be OS-specific)
   string16 version;
 
   // A description of the plugin that we get from its version info.
   string16 desc;
 
   // A list of all the mime types that this plugin supports.
   std::vector<WebPluginMimeType> mime_types;
 
   // Whether the plugin is enabled.
   bool enabled;
+
+  // Reason for the plugin being either enabled or disabled.
+  int reason;
+
+  // Priority of the plugin (obsolete?)
+  int priority;
 };
 
-#endif  // WEBKIT_GLUE_WEBPLUGININFO_H_
+#endif  // WEBKIT_GLUE_PLUGINS_WEBPLUGININFO_H_