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

Unified Diff: webkit/plugins/npapi/plugin_group_unittest.cc

Issue 10388253: Revert 138502 - Move version metadata from PluginGroup into PluginInstaller. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 7 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
« no previous file with comments | « webkit/plugins/npapi/plugin_group.cc ('k') | webkit/plugins/npapi/plugin_list.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/plugins/npapi/plugin_group_unittest.cc
===================================================================
--- webkit/plugins/npapi/plugin_group_unittest.cc (revision 138513)
+++ webkit/plugins/npapi/plugin_group_unittest.cc (working copy)
@@ -8,14 +8,128 @@
#include <vector>
#include "base/memory/scoped_ptr.h"
+#include "base/string16.h"
+#include "base/string_util.h"
#include "base/utf_string_conversions.h"
+#include "base/values.h"
#include "base/version.h"
#include "testing/gtest/include/gtest/gtest.h"
+#include "webkit/plugins/webplugininfo.h"
namespace webkit {
namespace npapi {
-TEST(PluginGroupTest, VersionExtraction) {
+static const VersionRangeDefinition kPluginVersionRange[] = {
+ { "", "", "3.0.44" }
+};
+static const VersionRangeDefinition kPlugin3VersionRange[] = {
+ { "0", "4", "3.0.44" }
+};
+static const VersionRangeDefinition kPlugin4VersionRange[] = {
+ { "4", "5", "4.0.44" }
+};
+static const VersionRangeDefinition kPlugin34VersionRange[] = {
+ { "0", "4", "3.0.44" },
+ { "4", "5", "" }
+};
+
+static const PluginGroupDefinition kPluginDef = {
+ "myplugin", "MyPlugin", "MyPlugin", kPluginVersionRange,
+ arraysize(kPluginVersionRange) };
+static const PluginGroupDefinition kPluginDef3 = {
+ "myplugin-3", "MyPlugin 3", "MyPlugin", kPlugin3VersionRange,
+ arraysize(kPlugin3VersionRange) };
+static const PluginGroupDefinition kPluginDef4 = {
+ "myplugin-4", "MyPlugin 4", "MyPlugin", kPlugin4VersionRange,
+ arraysize(kPlugin4VersionRange) };
+static const PluginGroupDefinition kPluginDef34 = {
+ "myplugin-34", "MyPlugin 3/4", "MyPlugin", kPlugin34VersionRange,
+ arraysize(kPlugin34VersionRange) };
+static const PluginGroupDefinition kPluginDefNotVulnerable = {
+ "myplugin-latest", "MyPlugin", "MyPlugin", NULL, 0 };
+
+const PluginGroupDefinition kPluginDefinitions[] = {
+ kPluginDef,
+ kPluginDef3,
+ kPluginDef4,
+ kPluginDef34,
+ kPluginDefNotVulnerable,
+};
+
+// name, path, version, desc.
+static const WebPluginInfo kPluginNoVersion = WebPluginInfo(
+ ASCIIToUTF16("MyPlugin"), FilePath(FILE_PATH_LITERAL("myplugin.so.2.0.43")),
+ string16(), ASCIIToUTF16("MyPlugin version 2.0.43"));
+static const WebPluginInfo kPlugin2043 = WebPluginInfo(
+ ASCIIToUTF16("MyPlugin"), FilePath(FILE_PATH_LITERAL("myplugin.so.2.0.43")),
+ ASCIIToUTF16("2.0.43"), ASCIIToUTF16("MyPlugin version 2.0.43"));
+static const WebPluginInfo kPlugin3043 = WebPluginInfo(
+ ASCIIToUTF16("MyPlugin"), FilePath(FILE_PATH_LITERAL("myplugin.so.3.0.43")),
+ ASCIIToUTF16("3.0.43"), ASCIIToUTF16("MyPlugin version 3.0.43"));
+static const WebPluginInfo kPlugin3044 = WebPluginInfo(
+ ASCIIToUTF16("MyPlugin"), FilePath(FILE_PATH_LITERAL("myplugin.so.3.0.44")),
+ ASCIIToUTF16("3.0.44"), ASCIIToUTF16("MyPlugin version 3.0.44"));
+static const WebPluginInfo kPlugin3045 = WebPluginInfo(
+ ASCIIToUTF16("MyPlugin"), FilePath(FILE_PATH_LITERAL("myplugin.so.3.0.45")),
+ ASCIIToUTF16("3.0.45"), ASCIIToUTF16("MyPlugin version 3.0.45"));
+static const WebPluginInfo kPlugin3045r = WebPluginInfo(
+ ASCIIToUTF16("MyPlugin"), FilePath(FILE_PATH_LITERAL("myplugin.so.3.0.45")),
+ ASCIIToUTF16("3.0r45"), ASCIIToUTF16("MyPlugin version 3.0r45"));
+static const WebPluginInfo kPlugin4043 = WebPluginInfo(
+ ASCIIToUTF16("MyPlugin"), FilePath(FILE_PATH_LITERAL("myplugin.so.4.0.43")),
+ ASCIIToUTF16("4.0.43"), ASCIIToUTF16("MyPlugin version 4.0.43"));
+
+class PluginGroupTest : public testing::Test {
+ public:
+ static PluginGroup* CreatePluginGroup(
+ const PluginGroupDefinition& definition) {
+ return PluginGroup::FromPluginGroupDefinition(definition);
+ }
+ static PluginGroup* CreatePluginGroup(const WebPluginInfo& wpi) {
+ return PluginGroup::FromWebPluginInfo(wpi);
+ }
+};
+
+TEST_F(PluginGroupTest, PluginGroupMatch) {
+ scoped_ptr<PluginGroup> group(PluginGroupTest::CreatePluginGroup(
+ kPluginDef3));
+ EXPECT_TRUE(group->Match(kPlugin3045));
+ EXPECT_TRUE(group->Match(kPlugin3045r));
+ EXPECT_FALSE(group->Match(kPluginNoVersion));
+ group->AddPlugin(kPlugin3045);
+ EXPECT_FALSE(group->IsVulnerable(kPlugin3045));
+
+ group.reset(PluginGroupTest::CreatePluginGroup(kPluginDef));
+ EXPECT_FALSE(group->Match(kPluginNoVersion));
+}
+
+TEST_F(PluginGroupTest, PluginGroupMatchCorrectVersion) {
+ scoped_ptr<PluginGroup> group(PluginGroupTest::CreatePluginGroup(
+ kPluginDef3));
+ EXPECT_TRUE(group->Match(kPlugin2043));
+ EXPECT_TRUE(group->Match(kPlugin3043));
+ EXPECT_FALSE(group->Match(kPlugin4043));
+
+ group.reset(PluginGroupTest::CreatePluginGroup(kPluginDef4));
+ EXPECT_FALSE(group->Match(kPlugin2043));
+ EXPECT_FALSE(group->Match(kPlugin3043));
+ EXPECT_TRUE(group->Match(kPlugin4043));
+
+ group.reset(PluginGroupTest::CreatePluginGroup(kPluginDef34));
+ EXPECT_TRUE(group->Match(kPlugin2043));
+ EXPECT_TRUE(group->Match(kPlugin3043));
+ EXPECT_TRUE(group->Match(kPlugin4043));
+}
+
+TEST_F(PluginGroupTest, PluginGroupDefinition) {
+ for (size_t i = 0; i < arraysize(kPluginDefinitions); ++i) {
+ scoped_ptr<PluginGroup> def_group(
+ PluginGroupTest::CreatePluginGroup(kPluginDefinitions[i]));
+ ASSERT_TRUE(def_group.get() != NULL);
+ }
+}
+
+TEST_F(PluginGroupTest, VersionExtraction) {
// Some real-world plugin versions (spaces, commata, parentheses, 'r', oh my)
const char* versions[][2] = {
{ "7.6.6 (1671)", "7.6.6.1671" }, // Quicktime
@@ -32,9 +146,46 @@
for (size_t i = 0; i < arraysize(versions); i++) {
scoped_ptr<Version> version(PluginGroup::CreateVersionFromString(
ASCIIToUTF16(versions[i][0])));
- EXPECT_EQ(versions[i][1], version->GetString());
+ EXPECT_STREQ(versions[i][1], version->GetString().c_str());
}
}
+TEST_F(PluginGroupTest, IsVulnerable) {
+ // Adobe Reader 10
+ VersionRangeDefinition adobe_reader_version_range[] = {
+ { "10", "11", "" },
+ { "9", "10", "9.4.1" },
+ { "0", "9", "8.2.5" }
+ };
+ PluginGroupDefinition adobe_reader_plugin_def = {
+ "adobe-reader", "Adobe Reader", "Adobe Acrobat",
+ adobe_reader_version_range, arraysize(adobe_reader_version_range) };
+ WebPluginInfo adobe_reader_plugin(ASCIIToUTF16("Adobe Reader"),
+ FilePath(FILE_PATH_LITERAL("/reader.so")),
+ ASCIIToUTF16("10.0.0.396"),
+ ASCIIToUTF16("adobe reader 10"));
+ scoped_ptr<PluginGroup> group(PluginGroupTest::CreatePluginGroup(
+ adobe_reader_plugin_def));
+ group->AddPlugin(adobe_reader_plugin);
+ PluginGroup group_copy(*group); // Exercise the copy constructor.
+ EXPECT_FALSE(group_copy.IsVulnerable(adobe_reader_plugin));
+
+ // Silverlight 4
+ VersionRangeDefinition silverlight_version_range[] = {
+ { "0", "4", "3.0.50106.0" },
+ { "4", "5", "" }
+ };
+ PluginGroupDefinition silverlight_plugin_def = {
+ "silverlight", "Silverlight", "Silverlight", silverlight_version_range,
+ arraysize(silverlight_version_range) };
+ WebPluginInfo silverlight_plugin(ASCIIToUTF16("Silverlight"),
+ FilePath(FILE_PATH_LITERAL("/silver.so")),
+ ASCIIToUTF16("4.0.50917.0"),
+ ASCIIToUTF16("silverlight 4"));
+ group.reset(PluginGroupTest::CreatePluginGroup(silverlight_plugin_def));
+ group->AddPlugin(silverlight_plugin);
+ EXPECT_FALSE(PluginGroup(*group).IsVulnerable(silverlight_plugin));
+}
+
} // namespace npapi
} // namespace webkit
« no previous file with comments | « webkit/plugins/npapi/plugin_group.cc ('k') | webkit/plugins/npapi/plugin_list.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698