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

Unified Diff: lib/appSupport.js

Issue 4559243822759936: Issue 431/432 - Remove special handling for the $sitekey option (Closed)
Patch Set: Created Aug. 12, 2014, 9:30 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 | lib/contentPolicy.js » ('j') | lib/contentPolicy.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/appSupport.js
===================================================================
--- a/lib/appSupport.js
+++ b/lib/appSupport.js
@@ -862,7 +862,7 @@
if (!host)
return;
- if (Policy.isWhitelisted(location.spec))
+ if (Policy.isWindowWhitelisted(exports.getBrowser(window).contentWindow))
Wladimir Palant 2014/08/14 19:58:15 Please revert the changes to this file. Sitekeys s
Thomas Greiner 2014/08/29 15:57:29 Done.
UI.removeWhitelist(window);
else
UI.toggleFilter(Filter.fromText("@@||" + host + "^$document"));
@@ -902,7 +902,7 @@
if (!host)
return;
- if (host && Policy.isWhitelisted(location.spec))
+ if (host && Policy.isWindowWhitelisted(exports.getBrowser(window).contentWindow))
action = "enable_site";
else if (host)
action = "disable_site";
« no previous file with comments | « no previous file | lib/contentPolicy.js » ('j') | lib/contentPolicy.js » ('J')

Powered by Google App Engine
This is Rietveld