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

Unified Diff: safari/include.youtube.js

Issue 29340571: Issue 3687 - Add experimental support for Safari content blockers (Closed)
Patch Set: Addressed Nits Created May 18, 2016, 11: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 | « safari/ext/content.js ('k') | skin/popup.css » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: safari/include.youtube.js
diff --git a/safari/include.youtube.js b/safari/include.youtube.js
index 0ad0bc2ed4011681293b8d23dfd01c25a38c9352..8d151ef944954938969a23dbddae310ed0f82acb 100644
--- a/safari/include.youtube.js
+++ b/safari/include.youtube.js
@@ -19,6 +19,22 @@
if (document.domain != "www.youtube.com")
return;
+ var usingContentBlockerAPI = true;
+ try
+ {
+ if (safari.self.tab.canLoad(beforeLoadEvent,
+ {category: "request",
+ payload: {type: "prefs.get",
+ key: "safariContentBlocker"}}) != true)
+ usingContentBlockerAPI = false;
+ }
+ catch (e)
+ {
+ }
+
+ if (usingContentBlockerAPI)
+ return;
+
if (ext.backgroundPage.sendMessageSync({type: "filters.isPageWhitelisted"}))
return;
@@ -33,7 +49,7 @@
return pairs.join("&");
}
- function patchPlayer(player)
+ function patchPlayer(player)
{
var newPlayer = player.cloneNode(true);
var flashvarsChanged = false;
« no previous file with comments | « safari/ext/content.js ('k') | skin/popup.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld