Remove TODOs

src/vs/workbench/parts/terminal/electron-browser/terminalActions.ts

@@ -523,7 +523,6 @@ export class RunActiveFileInTerminalAction extends Action {
 	}
 }
 
-// TODO: Change name to tab
 export class SwitchTerminalAction extends Action {
 
 	public static readonly ID = 'workbench.action.terminal.switchTerminal';

src/vs/workbench/parts/terminal/electron-browser/terminalTab.ts

@@ -283,7 +283,6 @@ export class TerminalTab extends Disposable implements ITerminalTab {
 		this.setActiveInstanceByIndex(this._getIndexFromId(instance.id));
 	}
 
-	// TODO: This is duplicated in ITerminalService
 	private _getIndexFromId(terminalId: number): number {
 		let terminalIndex = -1;
 		this.terminalInstances.forEach((terminalInstance, i) => {
@@ -331,7 +330,6 @@ export class TerminalTab extends Disposable implements ITerminalTab {
 		if (this._tabElement) {
 			this._tabElement.style.display = visible ? '' : 'none';
 		}
-		// TODO: probably don't need to tell terminal instances about visiblility anymore?
 		this.terminalInstances.forEach(i => i.setVisible(visible));
 	}