Avoid onError call if context is lost after finalization (visgl#5992)

harisbal · Jul 21, 2021 · 8b71107 · 8b71107
1 parent de8136d
commit 8b71107
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/modules/core/src/lib/deck.js b/modules/core/src/lib/deck.js
@@ -508,7 +508,7 @@ export default class Deck {
           ...opts,
           canvas: this.canvas,
           debug,
-          onContextLost: _ => onError?.(new Error(`WebGL context is lost`))
+          onContextLost: () => this._onContextLost()
         }),
       onInitialize: context => this._setGLContext(context.gl),
       onRender: this._onRenderFrame.bind(this),
@@ -536,6 +536,13 @@ export default class Deck {
     return views;
   }
 
+  _onContextLost() {
+    const {onError} = this.props;
+    if (this.animationLoop && onError) {
+      onError(new Error(`WebGL context is lost`));
+    }
+  }
+
   // The `pointermove` event may fire multiple times in between two animation frames,
   // it's a waste of time to run picking without rerender. Instead we save the last pick
   // request and only do it once on the next animation frame.