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

Unified Diff: assets/js/start.js

Issue 8851049: ABP/Android Domain specific filters (Closed)
Patch Set: Created Nov. 15, 2012, 7:38 a.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 | res/values/bools.xml » ('j') | res/values/bools.xml » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: assets/js/start.js
===================================================================
--- a/assets/js/start.js
+++ b/assets/js/start.js
@@ -559,15 +559,18 @@
contentType = "OTHER";
if (refHost != "")
- {
thirdParty = isThirdParty(reqHost, refHost);
- }
if (query != "")
url = url + "?" + query;
var filter = defaultMatcher.matchesAny(url, contentType, null, thirdParty);
+ // hack: if there is no referrer block only if filter is domain-specific
+ // (to re-enable in-app ads blocking, proposed on 12.11.2012 Monday meeting)
+ if (filter != null && refHost == "" && filter.text.indexOf("||") != 0)
+ filter = null;
+
return (filter != null && !(filter instanceof WhitelistFilter));
}
« no previous file with comments | « no previous file | res/values/bools.xml » ('j') | res/values/bools.xml » ('J')

Powered by Google App Engine
This is Rietveld