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

Unified Diff: content/common/plugin_list_unittest.cc

Issue 19844003: Remove webkit/plugins/npapi. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: remove GetDefaultWindowParent Created 7 years, 5 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
Index: content/common/plugin_list_unittest.cc
===================================================================
--- content/common/plugin_list_unittest.cc (revision 212595)
+++ content/common/plugin_list_unittest.cc (working copy)
@@ -15,6 +15,10 @@
namespace {
+base::FilePath::CharType kFooPath[] = FILE_PATH_LITERAL("/plugins/foo.plugin");
+base::FilePath::CharType kBarPath[] = FILE_PATH_LITERAL("/plugins/bar.plugin");
+const char* kFooName = "Foo Plugin";
+
bool Equals(const WebPluginInfo& a, const WebPluginInfo& b) {
return (a.name == b.name &&
a.path == b.path &&
@@ -34,13 +38,6 @@
} // namespace
-// Linux Aura and Android don't support NPAPI.
-#if defined(OS_WIN) || defined(OS_MACOSX) || (defined(OS_LINUX) && !defined(USE_AURA))
-
-base::FilePath::CharType kFooPath[] = FILE_PATH_LITERAL("/plugins/foo.plugin");
-base::FilePath::CharType kBarPath[] = FILE_PATH_LITERAL("/plugins/bar.plugin");
-const char* kFooName = "Foo Plugin";
-
class PluginListTest : public testing::Test {
public:
PluginListTest()
@@ -68,7 +65,7 @@
TEST_F(PluginListTest, GetPlugins) {
std::vector<WebPluginInfo> plugins;
- plugin_list_.GetPlugins(&plugins);
+ plugin_list_.GetPlugins(&plugins, true);
EXPECT_EQ(2u, plugins.size());
EXPECT_TRUE(Contains(plugins, foo_plugin_));
EXPECT_TRUE(Contains(plugins, bar_plugin_));
@@ -83,12 +80,10 @@
// Now we should have them in the state we specified above.
plugin_list_.RefreshPlugins();
std::vector<WebPluginInfo> plugins;
- plugin_list_.GetPlugins(&plugins);
+ plugin_list_.GetPlugins(&plugins, true);
ASSERT_TRUE(Contains(plugins, plugin_3043));
}
-#endif
-
#if defined(OS_POSIX) && !defined(OS_MACOSX)
// Test parsing a simple description: Real Audio.

Powered by Google App Engine
This is Rietveld 408576698