Issue 3687 - Add experimental support for Safari content blockers

safari/include.youtube.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 10 matching lines @@
 
   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/17 18:35:26]

Nit: Redundant semicolon.

Also I wonder whether we should put those braces on new lines. I think our
coding style actually requires it, but I wouldn't insist.

(Both, same for the code in safari/ext/content.js)

[kzar, 2016/05/17 19:20:23]

Done.

+  catch (e)
+  {
+  }
 
   if (usingContentBlockerAPI)
     return;
 
   if (ext.backgroundPage.sendMessageSync({type: "filters.isPageWhitelisted"}))
     return;
 
   var badArgumentsRegex = /^((.*_)?(ad|ads|afv|adsense)(_.*)?|(ad3|st)_module|prerolls|interstitial|infringe|iv_cta_url)$/;
 
   function rewriteFlashvars(flashvars)
@@ skipping 128 matching lines @@
               config.args = rawConfig.args;
             }
           }
         });
 
         ytplayer.config = rawYtplayer.config;
       }
     });
   }, badArgumentsRegex);
 })();