Closed Bug 1498331 Opened 2 years ago Closed 2 years ago
Emitter .off doesn't support being called with more than 3 arguments
46 bytes, text/x-phabricator-request
|Details | Review|
I noticed this issue while working on the tests for the old performance panel: the event handlers are not removed where they should, because there's some generic code that calls `off` with 3 arguments (which translates to 4 arguments when the static method is called). Because of this the event handler is never removed.
I checked that the new test fails on central.
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/00388a3697f1 Supports the case where EventEmitter.off is called with more than 3 arguments r=jdescottes
You need to log in before you can comment on or make changes to this bug.