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

Unified Diff: webkit/tools/test_shell/test_shell.cc

Issue 8602002: Move some webkit_glue embedder functions into WebKitPlatformSupport virtual methods (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: correct review items Created 9 years, 1 month 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
Index: webkit/tools/test_shell/test_shell.cc
diff --git a/webkit/tools/test_shell/test_shell.cc b/webkit/tools/test_shell/test_shell.cc
index 324c0f35bdbce876e038970588e3518fd842054e..3c13295973cd45533431a0e1b0a0198954054d21 100644
--- a/webkit/tools/test_shell/test_shell.cc
+++ b/webkit/tools/test_shell/test_shell.cc
@@ -47,8 +47,6 @@
#include "webkit/glue/user_agent.h"
#include "webkit/glue/webkit_glue.h"
#include "webkit/glue/webpreferences.h"
-#include "webkit/plugins/npapi/plugin_list.h"
-#include "webkit/plugins/webplugininfo.h"
#include "webkit/tools/test_shell/notification_presenter.h"
#include "webkit/tools/test_shell/simple_resource_loader_bridge.h"
#include "webkit/tools/test_shell/test_navigation_controller.h"
@@ -640,26 +638,4 @@ bool IsProtocolSupportedForMedia(const GURL& url) {
return false;
}
-void GetPlugins(bool refresh,
- std::vector<webkit::WebPluginInfo>* plugins) {
- if (refresh)
- webkit::npapi::PluginList::Singleton()->RefreshPlugins();
- webkit::npapi::PluginList::Singleton()->GetPlugins(plugins);
- // Don't load the forked TestNetscapePlugIn in the chromium code, use
- // the copy in webkit.org's repository instead.
- const FilePath::StringType kPluginBlackList[] = {
- FILE_PATH_LITERAL("npapi_layout_test_plugin.dll"),
- FILE_PATH_LITERAL("WebKitTestNetscapePlugIn.plugin"),
- FILE_PATH_LITERAL("libnpapi_layout_test_plugin.so"),
- };
- for (int i = plugins->size() - 1; i >= 0; --i) {
- webkit::WebPluginInfo plugin_info = plugins->at(i);
- for (size_t j = 0; j < arraysize(kPluginBlackList); ++j) {
- if (plugin_info.path.BaseName() == FilePath(kPluginBlackList[j])) {
- plugins->erase(plugins->begin() + i);
- }
- }
- }
-}
-
} // namespace webkit_glue

Powered by Google App Engine
This is Rietveld 408576698