issue #290 - [seamonkey] blockable items list persists on tab changes

lib/appSupport.js

Index: lib/appSupport.js
===================================================================
--- a/lib/appSupport.js
+++ b/lib/appSupport.js
@@ -145,6 +145,11 @@
  */
 exports.addBrowserLocationListener = function addBrowserLocationListener(/**Window*/ window, /**Function*/ callback, /**Boolean*/ ignoreSameDoc)
 {
+  _addBrowserLocationListener(window, callback, ignoreSameDoc);
+};
+
+function _addBrowserLocationListener(/**Window*/ window, /**Function*/ callback, /**Boolean*/ ignoreSameDoc)
+{
   let browser = (exports.getBrowser ? exports.getBrowser(window) : null);
   if (browser)
   {
@@ -170,7 +175,7 @@
     else
       progressListeners.set(window, [progressListener]);
   }
-};
+}
 
 /**
  * Removes a location listener registered for a window.
@@ -359,6 +364,12 @@
         return (browser ? browser.currentURI : null);
       }
     };
+    
+    exports.addBrowserLocationListener = function addBrowserLocationListener(/**Window*/ window, /**Function*/ callback, /**Boolean*/ ignoreSameDoc)
+    {
+      // for Seamonkey we have to ignore same document flag because of bug #1035171

[Wladimir Palant, 2014/07/08 11:29:02]

Please use a link, otherwise it won't be obvious where to look up this bug
number (our bug database, Mozilla's, Chrome's).

[saroyanm, 2014/07/08 11:58:42]

Done.

+      _addBrowserLocationListener(window, callback, false);
+    };

[Wladimir Palant, 2014/07/08 11:29:02]

Forcing the ignoreSameDoc parameter to be always false seems to be a sensible
approach. However, I would prefer keeping the entire hack in the SeaMonkey
section then. Something along these lines:

  let origAddBrowserLocationListener = exports.addBrowserLocationListener;
  exports.addBrowserLocationListener = function
sm_addBrowserLocationListener(window, callback, ignoreSameDoc)
  {
    origAddBrowserLocationListener(window, callback, false);
  };

[saroyanm, 2014/07/08 11:58:42]

I like this :)

 
     exports.contentContextMenu = ["contentAreaContextMenu", "mailContext"];