Disable history API on YouTube in order to block ads in subsequent videos

safari/include.youtube.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2013 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
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 (function() {
   if (document.domain != "www.youtube.com")
     return;
-
-  pushStateDisabled = false;
 
   function rewriteFlashvars(flashvars)
   {
     var pairs = flashvars.split("&");
     for (var i = 0; i < pairs.length; i++)
       if (/^((ad|afv|adsense|iv)(_.*)?|(ad3|iv3|st)_module|prerolls|interstitial|infringe|invideo)=/.test(pairs[i]))
         pairs.splice(i--, 1);
     return pairs.join("&");
   }
 
@@ skipping 29 matching lines @@
     }
 
     if (flashvarsChanged)
       player.parentNode.replaceChild(newPlayer, player);
 
     // if history.pushState is available, YouTube uses the history API
     // when navigation from one video to another, and tells the flash
     // player with JavaScript which video and which ads to show next,
     // bypassing our flashvars rewrite code. So we disable
     // history.pushState on pages with YouTube's flash player.
-    if (!pushStateDisabled)
-    {
-      var script = document.createElement("script");
-      script.type = "application/javascript";
-      script.textContent = "history.pushState = undefined;";
-      document.documentElement.appendChild(script);

[Wladimir Palant, 2014/02/24 13:15:03]

We shouldn't leave this script around, it needs to be removed once it did its
job. The other option would be running location.href =
"javascript:void(history.pushState = undefined);" - this shouldn't leave any
artifacts behind.

[Sebastian Noack, 2014/02/24 14:32:09]

Done.

-
-      pushStateDisabled = true;

[Wladimir Palant, 2014/02/24 13:15:03]

Why is this flag necessary? Content scripts should never run twice for the same
page.

[Sebastian Noack, 2014/02/24 14:32:09]

Yes, but the code runs per player loaded. However now where we don't add a
script tag anymore, I also got rid of that flag.

-    }
+    document.location.href = "javascript:void(history.pushState = undefined);";
   }
 
   var deferred = [];
   function patchPlayerDeferred(player)
   {
     deferred.push(player);
   }
 
   var onBeforeLoadYoutubeVideo = patchPlayerDeferred;
   function onBeforeLoad(event)
   {
     if ((event.target.localName == "object" || event.target.localName == "embed") && /:\/\/[^\/]*\.ytimg\.com\//.test(event.url))
       onBeforeLoadYoutubeVideo(event.target);
   }
 
   ext.backgroundPage.sendMessage({type: "get-domain-enabled-state"}, function(response)
   {
     if (response.enabled)
     {
       deferred.forEach(patchPlayer);
       onBeforeLoadYoutubeVideo = patchPlayer;
     }
     else
       document.removeEventListener("beforeload", onBeforeLoad);
   });
 
   document.addEventListener("beforeload", onBeforeLoad, true);
 })();