Commit f0f96632 authored by ananta@chromium.org's avatar ananta@chromium.org

Fix a build break by removing some webkit_glue stub functions from plugin_test.cpp. These will

get deleted anyway by jam's pending media player plugin changes.

I also disabled the DefaultPluginParsingTest as it fails when we remove these stub functions. However
this is also getting deleted in jam's media player change.

TBR=amit, jam(FYI)

Review URL: http://codereview.chromium.org/201051

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@25648 0039d316-1c4b-4281-b951-d872f2087c98
parent df7fba1c
...@@ -249,27 +249,7 @@ TEST_F(ActiveXTest, DISABLED_EmbeddedMPE) { ...@@ -249,27 +249,7 @@ TEST_F(ActiveXTest, DISABLED_EmbeddedMPE) {
TestActiveX(L"mpe_test.html", kLongWaitTimeout, false); TestActiveX(L"mpe_test.html", kLongWaitTimeout, false);
} }
TEST_F(PluginTest, DISABLED_DefaultPluginParsingTest) {
// 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.
namespace webkit_glue {
bool DownloadUrl(const std::string& url, HWND caller_window) {
return false;
}
bool GetPluginFinderURL(std::string* plugin_finder_url) {
return true;
}
} // namespace webkit_glue
bool IsPluginProcess() {
return false;
}
TEST_F(PluginTest, DefaultPluginParsingTest) {
PluginInstallerImpl plugin_installer(NP_EMBED); PluginInstallerImpl plugin_installer(NP_EMBED);
NPP_t plugin_instance = {0}; NPP_t plugin_instance = {0};
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment