Skip to content

Commit 0911688

Browse files
authored
Merge pull request #5817 from juspay/fix/dispose-leaks
Fix dispose leaks: register CursorBlinkStateManager + _pausedResizeTask
2 parents 62ce318 + 7a6e724 commit 0911688

3 files changed

Lines changed: 6 additions & 2 deletions

File tree

addons/addon-webgl/src/WebglRenderer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ import { createRenderDimensions } from 'browser/renderer/shared/RendererUtils';
3030

3131
export class WebglRenderer extends Disposable implements IRenderer {
3232
private _renderLayers: IRenderLayer[];
33-
private _cursorBlinkStateManager: MutableDisposable<CursorBlinkStateManager> = new MutableDisposable();
33+
private _cursorBlinkStateManager: MutableDisposable<CursorBlinkStateManager> = this._register(new MutableDisposable());
3434
private _textBlinkStateManager: TextBlinkStateManager;
3535
private _charAtlasDisposable = this._register(new MutableDisposable());
3636
private _charAtlas: ITextureAtlas | undefined;

src/browser/services/RenderService.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ export class RenderService extends Disposable implements IRenderService {
6868
) {
6969
super();
7070

71-
this._pausedResizeTask = new DebouncedIdleTask(this._logService);
71+
this._pausedResizeTask = this._register(new DebouncedIdleTask(this._logService));
7272

7373
this._renderDebouncer = new RenderDebouncer((start, end) => this._renderRows(start, end), this._coreBrowserService);
7474
this._register(this._renderDebouncer);

src/common/TaskQueue.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,4 +168,8 @@ export class DebouncedIdleTask {
168168
public flush(): void {
169169
this._queue.flush();
170170
}
171+
172+
public dispose(): void {
173+
this._queue.clear();
174+
}
171175
}

0 commit comments

Comments
 (0)