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

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

Issue 29321487: Issue 2738 - Fix tests to pass type bit mask to matching functions (Closed)
Patch Set: Created July 9, 2015, 1:46 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 | « no previous file | chrome/content/tests/performance/matching.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/tests/matcher.js
diff --git a/chrome/content/tests/matcher.js b/chrome/content/tests/matcher.js
index 1387d888168c3ccf932877122cb0176e809ad444..bda1a8d934778663ef13ae431ed56acab5b538e9 100644
--- a/chrome/content/tests/matcher.js
+++ b/chrome/content/tests/matcher.js
@@ -30,7 +30,7 @@
for (let filter of filters)
matcher.add(Filter.fromText(filter));
- let result = matcher.matchesAny(location, contentType, docDomain, thirdParty, sitekey);
+ let result = matcher.matchesAny(location, RegExpFilter.toTypeMask(contentType), docDomain, thirdParty, sitekey);
if (result)
result = result.text;
@@ -42,7 +42,7 @@
for (let filter of filters)
combinedMatcher.add(Filter.fromText(filter));
- let result = combinedMatcher.matchesAny(location, contentType, docDomain, thirdParty, sitekey);
+ let result = combinedMatcher.matchesAny(location, RegExpFilter.toTypeMask(contentType), docDomain, thirdParty, sitekey);
if (result)
result = result.text;
@@ -57,7 +57,7 @@
function cacheCheck(matcher, location, contentType, docDomain, thirdParty, expected)
{
- let result = matcher.matchesAny(location, contentType, docDomain, thirdParty);
+ let result = matcher.matchesAny(location, RegExpFilter.toTypeMask(contentType), docDomain, thirdParty);
if (result)
result = result.text;
« no previous file with comments | « no previous file | chrome/content/tests/performance/matching.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld