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

Unified Diff: lib/contentPolicy.js

Issue 29329754: Issue 3251 - Delegate processing of element hiding hits to shouldAllowAsync() so that hits show up (Closed)
Patch Set: Created Nov. 4, 2015, 3 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 | « lib/child/elemHide.js ('k') | lib/elemHide.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/contentPolicy.js
===================================================================
--- a/lib/contentPolicy.js
+++ b/lib/contentPolicy.js
@@ -173,17 +173,17 @@ var Policy = exports.Policy =
if (frame == testSitekeyFrame)
[testSitekey, testSitekeyFrame] = getSitekey(frames.slice(i + 1));
}
}
if (!match && contentType == "ELEMHIDE")
{
- match = location;
+ match = ElemHide.getFilterByKey(location);
tschuster 2015/11/05 15:39:36 So this is better, but we should still document th
Wladimir Palant 2015/11/06 11:29:00 Done.
location = match.text.replace(/^.*?#/, '#');
if (!match.isActiveOnDomain(docDomain))
return response(true, false);
let exception = ElemHide.getException(match, docDomain);
if (exception)
{
« no previous file with comments | « lib/child/elemHide.js ('k') | lib/elemHide.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld