Potential bug?

Handler is never unset:

https://github.com/xtermjs/xterm.js/blob/bd0d267d972a1aab4777cddecf9bf8196d7aa44f/src/browser/Lifecycle.ts#L13-L29

	export function addDisposableDomListener(
	node: Element \| Window \| Document,
	type: string,
	handler: (e: any) => void,
	useCapture?: boolean
	): IDisposable {
	node.addEventListener(type, handler, useCapture);
	return {
	dispose: () => {
	if (!handler) {
	// Already disposed
	return;
	}
	node.removeEventListener(type, handler, useCapture);
	}
	};
	}

Provide feedback