Merge pull request #2616 from sdegutis/patch-1

Avoid roundtrip to browser when double-disposing.
This commit is contained in:
Daniel Imms
2019-12-06 16:18:30 -08:00
committed by GitHub
+3 -2
View File
@@ -17,12 +17,13 @@ export function addDisposableDomListener(
useCapture?: boolean
): IDisposable {
node.addEventListener(type, handler, useCapture);
let disposed = false;
return {
dispose: () => {
if (!handler) {
// Already disposed
if (!disposed) {
return;
}
disposed = true;
node.removeEventListener(type, handler, useCapture);
}
};