Avoid roundtrip to browser when double-disposing.

This commit is contained in:
Steven Degutis
2019-12-06 15:57:26 -06:00
committed by GitHub
parent bd0d267d97
commit c5e144f327
+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);
}
};