Unified Diff: safari/include.youtube.js
Issue 29340571:
Issue 3687 - Add experimental support for Safari content blockers (Closed)
Patch Set: Remove onChange logic for checkboxes
Created May 18, 2016, 10:43 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« safari/ext/content.js ('K')
|
« safari/ext/content.js ('k')
|
skin/popup.css » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: safari/include.youtube.js |
| diff --git a/safari/include.youtube.js b/safari/include.youtube.js |
| index 0ad0bc2ed4011681293b8d23dfd01c25a38c9352..4fdacd1922690fe417d8ef65b8240a6621de1ca4 100644 |
| --- a/safari/include.youtube.js |
| +++ b/safari/include.youtube.js |
| @@ -19,6 +19,21 @@ |
| 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) |
| + { } |
|
Sebastian Noack
2016/05/18 11:16:33
Nit: The closing brace should go on a new line as
kzar
2016/05/18 11:32:08
Done.
|
| + |
| + if (usingContentBlockerAPI) |
| + return; |
| + |
| if (ext.backgroundPage.sendMessageSync({type: "filters.isPageWhitelisted"})) |
| return; |
| @@ -33,7 +48,7 @@ |
| return pairs.join("&"); |
| } |
| - function patchPlayer(player) |
| + function patchPlayer(player) |
| { |
| var newPlayer = player.cloneNode(true); |
| var flashvarsChanged = false; |
« safari/ext/content.js ('K')
|
« safari/ext/content.js ('k')
|
skin/popup.css » ('j')
|
no next file with comments »
