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

Unified Diff: lib/contentPolicy.js

Issue 29340656: Issue 3960 - Console errors due to wrong variable name (Closed)
Patch Set: Created April 20, 2016, 2:04 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 | no next file » | 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
@@ -269,17 +269,17 @@ var Policy = exports.Policy =
let frame = frames[i];
let wndLocation = frame.location;
let parentWndLocation = frames[Math.min(i + 1, frames.length - 1)].location;
let parentDocDomain = getHostname(parentWndLocation);
let match = defaultMatcher.matchesAny(wndLocation, typeMap, parentDocDomain, false, sitekey);
if (match instanceof WhitelistFilter)
{
- let whitelistType = (whitelistMatch.contentType & RegExpFilter.typeMap.DOCUMENT) ? "DOCUMENT" : "ELEMHIDE";
+ let whitelistType = (match.contentType & RegExpFilter.typeMap.DOCUMENT) ? "DOCUMENT" : "ELEMHIDE";
return [i, whitelistType, parentDocDomain, false, wndLocation, match];
}
if (!nogenericHit)
{
match = defaultMatcher.matchesAny(wndLocation,
RegExpFilter.typeMap[genericType], parentDocDomain, false, sitekey);
if (match instanceof WhitelistFilter)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld