diff options
| author | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-19 21:01:47 +0000 |
|---|---|---|
| committer | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-19 21:01:47 +0000 |
| commit | f5cdaff1456dfd7a5ad2240115285010a32f7fc1 (patch) | |
| tree | 924c291f7e8f97faeb3bf2db3973d17d7e08da9d /webkit | |
| parent | 1eae973272945a9d601f8cb2e54d7417a714d41d (diff) | |
| download | chromium_src-f5cdaff1456dfd7a5ad2240115285010a32f7fc1.zip chromium_src-f5cdaff1456dfd7a5ad2240115285010a32f7fc1.tar.gz chromium_src-f5cdaff1456dfd7a5ad2240115285010a32f7fc1.tar.bz2 | |
Lookup the plugin for the mime type before attempting to open a channel for the plugin. This ensures that the plugin will always be instantiated with the correct mime type at all times.
Added plugin tests to cover this case where in the embed tag is specified with a URL but without a mime type.
Fixes bug http://code.google.com/p/chromium/issues/detail?id=8973
Bug=8973
Test :- Added a plugin test for this case.
Review URL: http://codereview.chromium.org/113326
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16414 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit')
| -rw-r--r-- | webkit/glue/webframeloaderclient_impl.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/webkit/glue/webframeloaderclient_impl.cc b/webkit/glue/webframeloaderclient_impl.cc index 5e22435..a1bed38 100644 --- a/webkit/glue/webframeloaderclient_impl.cc +++ b/webkit/glue/webframeloaderclient_impl.cc @@ -1472,9 +1472,6 @@ Widget* WebFrameLoaderClient::createPlugin(const IntSize& size, // TODO(erikkay) // Attempt to map this clsid to a known NPAPI mime type if possible, failing // which we attempt to load the activex shim for the clsid. if (!activex_shim::GetMimeTypeForClsid(clsid, &my_mime_type)) { - // We only allowed specific ActiveX controls to run from certain websites. - if (!activex_shim::IsActiveXAllowed(clsid, url)) - return NULL; // We need to pass the combined clsid + version to PluginsList, so that it // would detect if the requested version is installed. If not, it needs // to use the default plugin to update the control. |