未验证 提交 515d4ef9 编写于 作者: D Daniel Imms 提交者: GitHub

Merge pull request #66186 from Microsoft/windowInNodeUsage

Don't use 'window' in node/ files
......@@ -19,7 +19,7 @@ export class TerminalProcess implements ITerminalChildProcess, IDisposable {
private _currentTitle: string = '';
private _processStartupComplete: Promise<void>;
private _isDisposed: boolean = false;
private _titleInterval: number = -1;
private _titleInterval: NodeJS.Timer | null = null;
private readonly _onProcessData = new Emitter<string>();
public get onProcessData(): Event<string> { return this._onProcessData.event; }
......@@ -91,8 +91,10 @@ export class TerminalProcess implements ITerminalChildProcess, IDisposable {
public dispose(): void {
this._isDisposed = true;
window.clearInterval(this._titleInterval);
this._titleInterval = -1;
if (this._titleInterval) {
clearInterval(this._titleInterval);
}
this._titleInterval = null;
this._onProcessData.dispose();
this._onProcessExit.dispose();
this._onProcessIdReady.dispose();
......@@ -105,7 +107,7 @@ export class TerminalProcess implements ITerminalChildProcess, IDisposable {
this._sendProcessTitle();
}, 0);
// Setup polling
this._titleInterval = window.setInterval(() => {
this._titleInterval = setInterval(() => {
if (this._currentTitle !== this._ptyProcess.process) {
this._sendProcessTitle();
}
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册