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: Rebased Created May 17, 2016, 9:19 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..a5c29dd133af57e4afbf7e719115e08c85760090 100644
--- a/safari/include.youtube.js
+++ b/safari/include.youtube.js
@@ -19,6 +19,17 @@
if (document.domain != "www.youtube.com")
return;
+ var usingContentBlockerAPI = true;
+ try {
+ if (safari.self.tab.canLoad(beforeLoadEvent,
+ {category: "usingContentBlockerAPI"}) != true)
+ usingContentBlockerAPI = false;
+ }
+ catch (e) { };
+
+ if (usingContentBlockerAPI)
+ return;
+
if (ext.backgroundPage.sendMessageSync({type: "filters.isPageWhitelisted"}))
return;
@@ -33,7 +44,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