eiac/xterm.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

fix(browser): reset TimeBasedDebouncer state on dispose

Browse Source 
dispose() cleared the pending timeout but left _refreshTimeoutID and
_additionalRefreshRequested set. After dispose, a stale timeout id or
trailing-refresh flag could confuse a new instance or tests. Match
RenderDebouncer by clearing the timeout id and reset the trailing flag.

Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
Daniel Imms
2026-06-01 15:27:58 +00:00
committed by Cursor Agent
co-authored by Cursor
parent 8679eba407
commit 275d6d8a80
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/browser/TimeBasedDebouncer.ts
+2
View File
@@ -31,7 +31,9 @@ export class TimeBasedDebouncer implements IRenderDebouncer {
public dispose(): void { public dispose(): void {
if (this._refreshTimeoutID) { if (this._refreshTimeoutID) {
clearTimeout(this._refreshTimeoutID); clearTimeout(this._refreshTimeoutID);
this._refreshTimeoutID = undefined;
} }
this._additionalRefreshRequested = false;
} }
public refresh(rowStart: number | undefined, rowEnd: number | undefined, rowCount: number): void { public refresh(rowStart: number | undefined, rowEnd: number | undefined, rowCount: number): void {