Merge pull request #292151 from microsoft/tyriar/291991

Fix Monaco scroll offset buffer use after destroy

Author: Tyriar

src/vs/editor/browser/gpu/renderStrategy/fullFileRenderStrategy.ts

@@ -172,6 +172,9 @@ export class FullFileRenderStrategy extends BaseRenderStrategy {
 	}
 
 	public override onScrollChanged(e?: ViewScrollChangedEvent): boolean {
+		if (this._store.isDisposed) {
+			return false;
+		}
 		const dpr = getActiveWindow().devicePixelRatio;
 		this._scrollOffsetValueBuffer[0] = (e?.scrollLeft ?? this._context.viewLayout.getCurrentScrollLeft()) * dpr;
 		this._scrollOffsetValueBuffer[1] = (e?.scrollTop ?? this._context.viewLayout.getCurrentScrollTop()) * dpr;

src/vs/editor/browser/gpu/renderStrategy/viewportRenderStrategy.ts

@@ -157,6 +157,9 @@ export class ViewportRenderStrategy extends BaseRenderStrategy {
 	}
 
 	public override onScrollChanged(e?: ViewScrollChangedEvent): boolean {
+		if (this._store.isDisposed) {
+			return false;
+		}
 		const dpr = getActiveWindow().devicePixelRatio;
 		this._scrollOffsetValueBuffer[0] = (e?.scrollLeft ?? this._context.viewLayout.getCurrentScrollLeft()) * dpr;
 		this._scrollOffsetValueBuffer[1] = (e?.scrollTop ?? this._context.viewLayout.getCurrentScrollTop()) * dpr;