Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(324)

Unified Diff: webkit/glue/plugins/plugin_list.cc

Issue 200031: Take out the activex control. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webkit/glue/plugins/plugin_list.h ('k') | webkit/glue/plugins/plugin_list_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/plugins/plugin_list.cc
===================================================================
--- webkit/glue/plugins/plugin_list.cc (revision 25626)
+++ webkit/glue/plugins/plugin_list.cc (working copy)
@@ -10,15 +10,11 @@
#include "base/time.h"
#include "net/base/mime_util.h"
#include "webkit/default_plugin/plugin_main.h"
+#include "webkit/glue/plugins/plugin_constants_win.h"
#include "webkit/glue/plugins/plugin_lib.h"
#include "webkit/glue/webkit_glue.h"
#include "googleurl/src/gurl.h"
-#if defined(OS_WIN)
-#include "webkit/activex_shim/activex_shared.h"
-#include "webkit/glue/plugins/plugin_constants_win.h"
-#endif
-
namespace NPAPI {
base::LazyInstance<PluginList> g_singleton(base::LINKER_INITIALIZED);
@@ -172,8 +168,6 @@
LoadPluginsFromDir(directories_to_scan[i], &new_plugins);
}
- LoadInternalPlugins(&new_plugins);
-
if (webkit_glue::IsDefaultPluginEnabled())
LoadPlugin(FilePath(kDefaultPluginLibraryName), &new_plugins);
@@ -216,7 +210,6 @@
}
bool PluginList::FindPlugin(const std::string& mime_type,
- const std::string& clsid,
bool allow_wildcard,
WebPluginInfo* info) {
DCHECK(mime_type == StringToLowerASCII(mime_type));
@@ -225,14 +218,6 @@
AutoLock lock(lock_);
for (size_t i = 0; i < plugins_.size(); ++i) {
if (SupportsType(plugins_[i], mime_type, allow_wildcard)) {
-#if defined(OS_WIN)
- if (!clsid.empty() && plugins_[i].path.value() == kActiveXShimFileName) {
- // Special handling for ActiveX shim. If ActiveX is not installed, we
- // should use the default plugin to show the installation UI.
- if (!activex_shim::IsActiveXInstalled(clsid))
- continue;
- }
-#endif
*info = plugins_[i];
return true;
}
@@ -241,7 +226,8 @@
return false;
}
-bool PluginList::FindPlugin(const GURL &url, std::string* actual_mime_type,
+bool PluginList::FindPlugin(const GURL &url,
+ std::string* actual_mime_type,
WebPluginInfo* info) {
LoadPlugins(false);
AutoLock lock(lock_);
@@ -308,19 +294,11 @@
bool PluginList::GetPluginInfo(const GURL& url,
const std::string& mime_type,
- const std::string& clsid,
bool allow_wildcard,
WebPluginInfo* info,
std::string* actual_mime_type) {
- bool found = FindPlugin(mime_type,
- clsid,
- allow_wildcard, info);
- if (!found
- || (info->path.value() == kDefaultPluginLibraryName
-#if defined(OS_WIN)
- && clsid.empty()
-#endif
- )) {
+ bool found = FindPlugin(mime_type, allow_wildcard, info);
+ if (!found || (info->path.value() == kDefaultPluginLibraryName)) {
WebPluginInfo info2;
if (FindPlugin(url, actual_mime_type, &info2)) {
found = true;
« no previous file with comments | « webkit/glue/plugins/plugin_list.h ('k') | webkit/glue/plugins/plugin_list_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698