summaryrefslogtreecommitdiffstats
path: root/chrome/test/plugin/plugin_test.cpp
diff options
context:
space:
mode:
authorananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-30 23:05:11 +0000
committerananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-30 23:05:11 +0000
commit6165e2ab4af4c217e21c0ec167b7eced8fca491c (patch)
tree387a179dc41e1398bc050edf595288bfd1315a8e /chrome/test/plugin/plugin_test.cpp
parente580551a844f99c341876c7564a719ea07f07e30 (diff)
downloadchromium_src-6165e2ab4af4c217e21c0ec167b7eced8fca491c.zip
chromium_src-6165e2ab4af4c217e21c0ec167b7eced8fca491c.tar.gz
chromium_src-6165e2ab4af4c217e21c0ec167b7eced8fca491c.tar.bz2
Added audio/mpeg and video/mpeg to the list of mime types supported by the Activex shim for media player along with their associated extensions.
This fixes http://code.google.com/p/chromium/issues/detail?id=11054, which occurs with embed tags with a source URL and no mime type. In this case the URL extension is mp3, which is not claimed by any plugin although Quicktime and media player do handle it. It works in Firefox because the plugin instantiation code in FF correctly gets the mime type from the http stream and instantiates the plugin accordingly. Webkit attempts to instantiate the plugin from the embed tag with the source URL. This fails silently in Chrome and Safari displays an error message. With this fix the only user visible difference would be quicktime handling the mp3 in FF and IE, and media player handling it in Chrome. I also moved the media player shim to the end of the plugin list to ensure it does not take over the audio/mpeg and video/mpeg mime types if there is a a plugin which can handle it. Added plugin tests for the mp3 and mpe extensions. Bug=11054 Review URL: http://codereview.chromium.org/100180 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15008 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/plugin/plugin_test.cpp')
-rw-r--r--chrome/test/plugin/plugin_test.cpp9
1 files changed, 9 insertions, 0 deletions
diff --git a/chrome/test/plugin/plugin_test.cpp b/chrome/test/plugin/plugin_test.cpp
index 27bf7d6..0b8cc9a 100644
--- a/chrome/test/plugin/plugin_test.cpp
+++ b/chrome/test/plugin/plugin_test.cpp
@@ -225,6 +225,15 @@ TEST_F(ActiveXTest, DISABLED_CustomScripting) {
TestActiveX(L"activex_custom_scripting.html", kShortWaitTimeout, true);
}
+TEST_F(ActiveXTest, EmbeddedMP3) {
+ TestActiveX(L"mp3_test.html", kLongWaitTimeout, false);
+}
+
+TEST_F(ActiveXTest, EmbeddedMPE) {
+ TestActiveX(L"mpe_test.html", kLongWaitTimeout, false);
+}
+
+
// The default plugin tests defined below rely on the following webkit
// functions and the IsPluginProcess function which is defined in the global
// namespace. Stubbed these out for now.