Index: extensions/common/url_pattern_unittest.cc |
diff --git a/extensions/common/url_pattern_unittest.cc b/extensions/common/url_pattern_unittest.cc |
index 9618aeb16a175be5c0fdced0f59274fe49d51ecd..fe5eb01e8866be648e234ae339ba6ac05fe2f192 100644 |
--- a/extensions/common/url_pattern_unittest.cc |
+++ b/extensions/common/url_pattern_unittest.cc |
@@ -810,4 +810,18 @@ TEST(ExtensionURLPatternTest, Subset) { |
EXPECT_TRUE(StrictlyContains(pattern12, pattern13)); |
} |
+TEST(ExtensionURLPatternTest, MatchesSingleOrigin) { |
+ URLPattern pattern1(URLPattern::SCHEME_ALL); |
+ URLPattern pattern2(URLPattern::SCHEME_ALL); |
+ URLPattern pattern3(URLPattern::SCHEME_ALL); |
+ |
+ EXPECT_EQ(URLPattern::PARSE_SUCCESS, pattern1.Parse("http://*/*")); |
+ EXPECT_EQ(URLPattern::PARSE_SUCCESS, pattern2.Parse("http://*.google.com/*")); |
+ EXPECT_EQ(URLPattern::PARSE_SUCCESS, pattern3.Parse("http://www.google.com/*")); |
+ |
+ EXPECT_FALSE(pattern1.MatchesSingleOrigin()); |
+ EXPECT_FALSE(pattern2.MatchesSingleOrigin()); |
+ EXPECT_TRUE(pattern3.MatchesSingleOrigin()); |
not at google - send to devlin
2014/08/11 22:59:30
I think it's unnecessarily complex to be making th
gpdavis
2014/08/12 00:52:06
Shouldn't MatchesSingleOrigin also check the schem
not at google - send to devlin
2014/08/12 19:49:26
Yes it should check the scheme and port.
|
+} |
+ |
} // namespace |