Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/tests/regexpFilters_matching.js

Issue 5971196453584896: Issue 2503 - Inconsistent behavior: $document flag implied for exception rules with protocol includ… (Closed)
Patch Set: Created May 18, 2015, 5:51 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/content/tests/matcher.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/tests/regexpFilters_matching.js
===================================================================
--- a/chrome/content/tests/regexpFilters_matching.js
+++ b/chrome/content/tests/regexpFilters_matching.js
@@ -288,18 +288,18 @@
testMatch("abc$domain=foo.com,sitekey=foo-publickey", "http://abc/def", "IMAGE", "bar.com", true, "foo-publickey", false);
testMatch("abc$domain=~foo.com,sitekey=foo-publickey", "http://abc/def", "IMAGE", "foo.com", true, "foo-publickey", false);
testMatch("abc$domain=~foo.com,sitekey=foo-publickey", "http://abc/def", "IMAGE", "bar.com", true, "foo-publickey", true);
});
test("Exception rules", function()
{
testMatch("@@test", "http://test/", "DOCUMENT", null, false, null, false);
- testMatch("@@http://test*", "http://test/", "DOCUMENT", null, false, null, true);
- testMatch("@@ftp://test*", "ftp://test/", "DOCUMENT", null, false, null, true);
+ testMatch("@@http://test*", "http://test/", "DOCUMENT", null, false, null, false);
+ testMatch("@@ftp://test*", "ftp://test/", "DOCUMENT", null, false, null, false);
testMatch("@@test$document", "http://test/", "DOCUMENT", null, false, null, true);
testMatch("@@test$document,image", "http://test/", "DOCUMENT", null, false, null, true);
testMatch("@@test$~image", "http://test/", "DOCUMENT", null, false, null, false);
testMatch("@@test$~image,document", "http://test/", "DOCUMENT", null, false, null, true);
testMatch("@@test$document,~image", "http://test/", "DOCUMENT", null, false, null, true);
testMatch("@@test$document,domain=foo.com", "http://test/", "DOCUMENT", "foo.com", false, null, true);
testMatch("@@test$document,domain=foo.com", "http://test/", "DOCUMENT", "bar.com", false, null, false);
testMatch("@@test$document,domain=~foo.com", "http://test/", "DOCUMENT", "foo.com", false, null, false);
« no previous file with comments | « chrome/content/tests/matcher.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld