Issue 261 - FilterNotifier doesn`t support listeners removing themselves when triggered

chrome/content/tests/filterNotifier.js

 (function()
 {
   module("Filter notifier", {setup: prepareFilterComponents, teardown: restoreFilterComponents});
 
   let triggeredListeners = [];
   let listeners = [
     function(action, item) triggeredListeners.push(["listener1", action, item]),
     function(action, item) triggeredListeners.push(["listener2", action, item]),
     function(action, item) triggeredListeners.push(["listener3", action, item])
   ];
@@ skipping 39 matching lines @@
 
     FilterNotifier.removeListener(listener3);
     compareListeners("removeListener(listener3)", [listener2, listener1]);
 
     FilterNotifier.removeListener(listener1);
     compareListeners("removeListener(listener1)", [listener2]);
 
     FilterNotifier.removeListener(listener2);
     compareListeners("removeListener(listener2)", []);
   });
+
+  test("Removing listeners while being called", function()
+  {
+    let listener1 = function(){

[Thomas Greiner, 2014/04/07 08:30:49]

Nit: Move bracket to new line

+      listeners[0].apply(this, arguments);
+      FilterNotifier.removeListener(listener1);
+    };
+    let listener2 = listeners[1];
+    FilterNotifier.addListener(listener1);
+    FilterNotifier.addListener(listener2);
+
+    compareListeners("Initial call", [listener1, listener2]);
+    compareListeners("Subsequent calls", [listener2]);
+  })

[Thomas Greiner, 2014/04/07 08:30:49]

Nit: Missing semicolon

 })();