Hook up bg terminal listeners before attach

Tyriar · Tyriar · commit 649d6befa1fe · 2019-05-24T17:32:52.000-07:00
Part of microsoft#74228
diff --git a/src/vs/workbench/contrib/terminal/browser/terminalService.ts b/src/vs/workbench/contrib/terminal/browser/terminalService.ts
@@ -57,6 +57,7 @@ export abstract class TerminalService extends CommonTerminalService implements I
 				shell,
 				true);
 			this._backgroundedTerminalInstances.push(instance);
+			this._initInstanceListeners(instance);
 			return instance;
 		}
 		const terminalTab = this._instantiationService.createInstance(TerminalTab,
@@ -88,7 +89,6 @@ export abstract class TerminalService extends CommonTerminalService implements I
 		this._terminalTabs.push(terminalTab);
 		terminalTab.addDisposable(terminalTab.onDisposed(this._onTabDisposed.fire, this._onTabDisposed));
 		terminalTab.addDisposable(terminalTab.onInstancesChanged(this._onInstancesChanged.fire, this._onInstancesChanged));
-		this._initInstanceListeners(instance);
 		if (this.terminalInstances.length === 1) {
 			// It's the first instance so it should be made active automatically
 			this.setActiveInstanceByIndex(0);
