Remove tabs from TabMap onLoading instead of onBeforeNavigate on Safari

chrome/ext/background.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
@@ skipping 269 matching lines @@
     activate: function()
     {
       chrome.tabs.update(this._id, {selected: true});
     },
     sendMessage: function(message, responseCallback)
     {
       sendMessage(this._id, message, responseCallback);
     }
   };
 
-  TabMap = function(deleteTabOnBeforeNavigate)
+  TabMap = function(deleteOnPageUnload)
   {
     this._map = {};
 
     this._delete = this._delete.bind(this);
-    this._deleteTabOnBeforeNavigate = deleteTabOnBeforeNavigate;
+    this._deleteOnPageUnload = deleteOnPageUnload;
   };
   TabMap.prototype = {
     get: function(tab)
     {
       return (this._map[tab._id] || {}).value;
     },
     set: function(tab, value)
     {
       if (!(tab._id in this._map))
       {
         tab.onRemoved.addListener(this._delete);
-        if (this._deleteTabOnBeforeNavigate)
+        if (this._deleteOnPageUnload)
           tab._onBeforeNavigate.addListener(this._delete);
       }
 
       this._map[tab._id] = {tab: tab, value: value};
     },
     has: function(tab)
     {
       return tab._id in this._map;
     },
     clear: function()
     {
       for (var id in this._map)
         this.delete(this._map[id].tab);
     },
     _delete: function(tab)
     {
       // delay so that other event handlers can still lookup this tab
       setTimeout(this.delete.bind(this, tab), 0);
-    }
-  };
-  TabMap.prototype["delete"] = function(tab)
-  {
-    delete this._map[tab._id];
-
-    tab.onRemoved.removeListener(this._delete);
-    tab._onBeforeNavigate.removeListener(this._delete);
+    },
+    delete: function(tab)
+    {
+      delete this._map[tab._id];
+
+      tab.onRemoved.removeListener(this._delete);
+      tab._onBeforeNavigate.removeListener(this._delete);
+    }
   };
 
 
   /* Windows */
 
   Window = function(win)
   {
     this._id = win.id;
     this.visible = win.status != "minimized";
   };
@@ skipping 233 matching lines @@
       if (isContextMenuHidden)
         return;
 
       chrome.contextMenus.removeAll();
       isContextMenuHidden = true;
     }
   };
 
   ext.onMessage = new BackgroundMessageEventTarget();
 })();