Fixed reload issue in Chrome

safari/content.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/>.
  */
 
+safari.self.tab.dispatchMessage("loading", document.location.href);

[Felix Dahlke, 2013/11/15 13:28:38]

Why not move this into the IIFE?

[Sebastian Noack, 2013/11/15 13:45:29]

As you wish. I didn't put it into the IIFE in the first place, because of that
code doesn't define variables and the only purpose of the IIFE below is to hide
variables, that don't need to be global.

[Felix Dahlke, 2013/11/15 13:48:49]

Oh, I actually meant "under" the IIFE (whether it's in there or not, we
typically have definitions first, then code.)

But it actually doesn't matter, it's up to you.

+
 (function()
 {
   /* Background page proxy */
   var proxy = {
     objects: [],
     callbacks: [],
 
     send: function(message)
     {
       var evt = document.createEvent("Event");
@@ skipping 264 matching lines @@
   ext.backgroundPage = {
     _eventTarget: safari.self,
     _messageDispatcher: safari.self.tab,
 
     sendMessage: sendMessage,
     getWindow: function() { return proxy.getObject(0); }
   };
 
   ext.onMessage = new MessageEventTarget(safari.self);
 })();