disable listener leak monitor for stable, #64493

src/vs/base/common/event.ts

@@ -129,7 +129,7 @@ export class Emitter<T> {
 	private static readonly _noop = function () { };
 
 	private readonly _options: EmitterOptions | undefined;
-	private readonly _leakageMon: LeakageMonitor;
+	private readonly _leakageMon: LeakageMonitor | undefined;
 	private _disposed: boolean = false;
 	private _event: Event<T> | undefined;
 	private _deliveryQueue: [Listener, (T | undefined)][] | undefined;
@@ -137,7 +137,9 @@ export class Emitter<T> {
 
 	constructor(options?: EmitterOptions) {
 		this._options = options;
-		this._leakageMon = new LeakageMonitor(this._options && this._options.leakWarningThreshold);
+		this._leakageMon = _globalLeakWarningThreshold > 0
+			? new LeakageMonitor(this._options && this._options.leakWarningThreshold)
+			: undefined;
 	}
 
 	/**
@@ -168,13 +170,16 @@ export class Emitter<T> {
 				}
 
 				// check and record this emitter for potential leakage
-				const check = this._leakageMon.check(this._listeners.size);
+				let removeMonitor: (() => void) | undefined;
+				if (this._leakageMon) {
+					removeMonitor = this._leakageMon.check(this._listeners.size);
+				}
 
 				let result: IDisposable;
 				result = {
 					dispose: () => {
-						if (check) {
-							check();
+						if (removeMonitor) {
+							removeMonitor();
 						}
 						result.dispose = Emitter._noop;
 						if (!this._disposed) {
@@ -238,7 +243,9 @@ export class Emitter<T> {
 		if (this._deliveryQueue) {
 			this._deliveryQueue.length = 0;
 		}
-		this._leakageMon.dispose();
+		if (this._leakageMon) {
+			this._leakageMon.dispose();
+		}
 		this._disposed = true;
 	}
 }

src/vs/workbench/electron-browser/main.ts

@@ -61,7 +61,8 @@ export function startup(configuration: IWindowConfiguration): Promise<void> {
 	perf.importEntries(configuration.perfEntries);
 
 	// Configure emitter leak warning threshold
-	setGlobalLeakWarningThreshold(100);
+	setGlobalLeakWarningThreshold(-1);
+
 
 	// Browser config
 	browser.setZoomFactor(webFrame.getZoomFactor()); // Ensure others can listen to zoom level changes