Bug in DomEvent: removeListener does not remove listener is some cases

[Scalar4eg]

removeListener does not remove SAME listeners
Here is simple test:

function OnClick() {console.log("i'm OnClick listener");}

for(var i=0;i<10;i++) L.DomEvent.addListener(document, "click", OnClick);
for(var i=0;i<10;i++) L.DomEvent.removeListener(document, "click", OnClick);

Now we can click on the page and we'll see message "i'm OnClick listener" 9 times instead of 0.

[mourner]

Hmm, I guess it shouldn't add the same listeners in the first place, right?

[anru]

From http://www.w3.org/TR/DOM-Level-2-Events/events.html#Events-Registration-interfaces-h3 :

If multiple identical EventListeners are registered on the same EventTarget with the same parameters the duplicate instances are discarded. They do not cause the EventListener to be called twice and since they are discarded they do not need to be removed with the removeEventListener method.

[mourner]

In this case Leaflet addListener creates a closure to bind it to the given context, so the listeners are not identical. We need to track this manually.

[mikila85]

so how am i spose to do this?
i get duplicate events fired because it initialized the controller few times :(