diff --git a/addons/addon-webgl/src/GlyphRenderer.ts b/addons/addon-webgl/src/GlyphRenderer.ts index be1990c2a..1289adec2 100644 --- a/addons/addon-webgl/src/GlyphRenderer.ts +++ b/addons/addon-webgl/src/GlyphRenderer.ts @@ -100,6 +100,7 @@ export class GlyphRenderer extends Disposable { private readonly _attributesBuffer: WebGLBuffer; private _atlas: ITextureAtlas | undefined; + private _lastAtlasPagesVersion: number = -1; private _activeBuffer: number = 0; private readonly _vertices: IVertices = { count: 0, @@ -214,7 +215,14 @@ export class GlyphRenderer extends Disposable { } public beginFrame(): boolean { - return this._atlas ? this._atlas.beginFrame() : true; + if (!this._atlas) { + return true; + } + if (this._atlas.pagesVersion !== this._lastAtlasPagesVersion) { + this._lastAtlasPagesVersion = this._atlas.pagesVersion; + return true; + } + return false; } public updateCell(x: number, y: number, code: number, bg: number, fg: number, ext: number, chars: string, width: number, lastBg: number): void { @@ -375,6 +383,7 @@ export class GlyphRenderer extends Disposable { public setAtlas(atlas: ITextureAtlas): void { this._atlas = atlas; + this._lastAtlasPagesVersion = -1; this.invalidateAtlasTextures(); } diff --git a/addons/addon-webgl/src/TextureAtlas.ts b/addons/addon-webgl/src/TextureAtlas.ts index 1c3e08a1c..73056b87e 100644 --- a/addons/addon-webgl/src/TextureAtlas.ts +++ b/addons/addon-webgl/src/TextureAtlas.ts @@ -132,12 +132,8 @@ export class TextureAtlas implements ITextureAtlas { } } - private _requestClearModel = false; - public beginFrame(): boolean { - const result = this._requestClearModel; - this._requestClearModel = false; - return result; - } + private _pagesVersion = 0; + public get pagesVersion(): number { return this._pagesVersion; } public clearTexture(): void { if (this._pages[0].currentRow.x === 0 && this._pages[0].currentRow.y === 0) { @@ -206,8 +202,8 @@ export class TextureAtlas implements ITextureAtlas { // Add the new merged page to the end this.pages.push(mergedPage); - // Request the model to be cleared to refresh all texture pages. - this._requestClearModel = true; + // Invalidate renderer models so all texture pages are refreshed. + this._pagesVersion++; this._onAddTextureAtlasCanvas.fire(mergedPage.canvas); } @@ -818,8 +814,8 @@ export class TextureAtlas implements ITextureAtlas { this._overflowSizePage = new AtlasPage(this._document, this._config.deviceMaxTextureSize); this.pages.push(this._overflowSizePage); - // Request the model to be cleared to refresh all texture pages. - this._requestClearModel = true; + // Invalidate renderer models so all texture pages are refreshed. + this._pagesVersion++; this._onAddTextureAtlasCanvas.fire(this._overflowSizePage.canvas); } activePage = this._overflowSizePage; diff --git a/addons/addon-webgl/src/Types.ts b/addons/addon-webgl/src/Types.ts index 294582e37..3a85eb4d0 100644 --- a/addons/addon-webgl/src/Types.ts +++ b/addons/addon-webgl/src/Types.ts @@ -68,10 +68,9 @@ export interface ITextureAtlas extends IDisposable { warmUp(): void; /** - * Call when a frame is being drawn, this will return true if the atlas was cleared to make room - * for a new set of glyphs. + * Incremented whenever texture page indexes or coordinates cached by renderers may be stale. */ - beginFrame(): boolean; + readonly pagesVersion: number; /** * Clear all glyphs from the texture atlas.