Bug 218305

Summary: WebGL context remains in memory after calling loseContext()
Product: WebKit Reporter: Philip <philipbell>
Component: WebGLAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: dino, jdarpinian, kbr, kkinnunen, maggotfish, noam, smoley, takashi.th.hasegawa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari 14   
Hardware: Unspecified   
OS: Unspecified   

Description Philip 2020-10-28 13:12:46 PDT
To reproduce: 
Create over 16 WebGL contexts and call loseContext on each.
On the 17th call two errors will be thrown: 
> ‘There are too many active WebGL contexts on this page, the oldest context will be lost.’
> ‘WebGL: INVALID_OPERATION: loseContext: context already lost’
It's expected that these contexts would not remain in memory.

Code snippet:
for (let i = 0; i < 17; i++) {
  const canvas = document.createElement('canvas');
  const gl = canvas.getContext('webgl');  
  const extension = gl.getExtension('WEBGL_lose_context')
  extension.loseContext();
}

Example CodePen of dynamically losing & restoring contexts.
https://codepen.io/philipbell/pen/RwRQGmo
Comment 1 Smoley 2020-10-29 18:04:48 PDT
Thanks for filing! Apple Internal see rdar://35920437.
Comment 2 Radar WebKit Bug Importer 2020-10-29 18:35:13 PDT
<rdar://problem/70838378>
Comment 3 Philip 2020-11-03 10:25:41 PST
Some additional context:
loseContext is used in popular WebGL libraries including Three.js and PixiJS. 
Without this method their context management code fails.

Three.js manages context directly in the renderer code:
https://github.com/mrdoob/three.js/blob/3989bb4a00a9f38b1df5dc02bcbc1e5a1b80df53/src/renderers/WebGLRenderer.js#L345-L357

PixiJS has a component dedicated to context management:
https://github.com/pixijs/pixi.js/blob/9ff49a23670bdb0ed864d4e53fb91d524eab59f2/packages/core/src/context/ContextSystem.ts#L248-L262
Comment 4 Yehonatan Daniv 2021-05-24 05:46:05 PDT
Looks like a duplicate of this: https://bugs.webkit.org/show_bug.cgi?id=200031
At Wix we use Kampos (https://github.com/wix/kampos), which doesn't use the WEBGL_lose_context plugin, but still getting the same error.