250307 2023-01-08 21:01:07 -0800 "TypeError: Type error" when calling WebGL bufferData with typed array backed by growable SharedArrayBuffer 2023-01-09 15:33:43 -0800 1 1 1 Unclassified WebKit WebGL Safari Technology Preview Mac (Apple Silicon) macOS 13 RESOLVED DUPLICATE 249635 https://bugs.webkit.org/show_bug.cgi?id=248024 InRadar P2 Normal --- 248024 1 Ian.kettlewell ysuzuki dino kbr kkinnunen webkit-bug-importer ysuzuki oldest_to_newest 1923987 0 464415 Ian.kettlewell 2023-01-08 21:01:07 -0800 Created attachment 464415 Index.html file with the code to reproduce the issue As title says. The following code errors in Safari Technical Preview 160: ``` let gl = canvas.getContext('webgl2'); const sab = new SharedArrayBuffer(1024, { maxByteLength: 2048 // This is important, without it the page runs fine }); const data = new Uint8Array(sab, 0, 32); let buffer = gl.createBuffer(); gl.bindBuffer(gl.ARRAY_BUFFER, buffer); // TypeError: Type error gl.bufferData(gl.ARRAY_BUFFER, data, gl.STATIC_DRAW); ``` I discovered this in a game jam entry I made (https://ianjk.com/ld52/) that I was testing in the Safari Technology Preview. For some reason the SharedArrayBuffer backing the WebAssembly binary is automatically made 'growable' in Safari Technology Preview 160. In Safari 16.2 that does not happen so both the attached example and my game jam entry run fine. I'm not versed in where to file WebKit bugs so I've filed this under 'WebGL', but please reclassify if there's a more appropriate 'Component'. :) 1924007 1 464416 kkinnunen 2023-01-09 01:43:46 -0800 Created attachment 464416 server to run the file with 1924008 2 kkinnunen 2023-01-09 01:45:23 -0800 Likely regressed with bug 248024 Yusuke, it appears intentional. Is there anything WebGL code can do to opt into using the growable array buffers? 1924009 3 webkit-bug-importer 2023-01-09 01:45:34 -0800 <rdar://problem/104022514> 1924066 4 ysuzuki 2023-01-09 06:00:18 -0800 (In reply to Kimmo Kinnunen from comment #2) > Likely regressed with bug 248024 > > Yusuke, it appears intentional. > Is there anything WebGL code can do to opt into using the growable array > buffers? Growable SharedArrayBuffer / Resizable ArrayBuffer cannot be used for IDL saying ArrayBuffer without [[AllowResizable]] annotation. (This is explicitly specified for resizable array-buffer / growable shared-array-buffer). https://webidl.spec.whatwg.org/#AllowResizable "If the [AllowResizable] extended attribute and the [AllowShared] extended attribute both appear on one of the buffer source types, it creates a new IDL type that allows the buffer source type to be additionally backed by an ECMAScript SharedArrayBuffer that is growable." Thus, given that WebGL IDL is not saying [[AllowResizable]], I think this behavior is expected. (bufferData does not have [[AllowResizable]]). On the other hand, > I discovered this in a game jam entry I made (https://ianjk.com/ld52/) that I was testing in the Safari Technology Preview. For some reason the SharedArrayBuffer backing the WebAssembly binary is automatically made 'growable' in Safari Technology Preview 160. In Safari 16.2 that does not happen so both the attached example and my game jam entry run fine. This is fixed in bug 249635. *** This bug has been marked as a duplicate of bug 249635 *** 1924188 5 kbr 2023-01-09 15:33:43 -0800 For what it's worth I think the WebGL IDL should be updated to add [AllowResizable] for compatibility with WebAssembly applications that use resizable memory. Have filed https://github.com/KhronosGroup/WebGL/issues/3510 about this. 464415 2023-01-08 21:01:07 -0800 2023-01-08 21:01:07 -0800 Index.html file with the code to reproduce the issue index.html text/html 523 Ian.kettlewell PGh0bWw+CjxoMT5IZWxsbzwvaDE+CjxjYW52YXMgaWQ9ImNhbnZhcyI+PC9jYW52YXM+Cgo8Ym9k eT4KICAgIDxzY3JpcHQ+CiAgICAgICAgY2FudmFzID0gZG9jdW1lbnQKICAgICAgICAgICAgLmdl dEVsZW1lbnRCeUlkKCJjYW52YXMiKTsKICAgICAgICBsZXQgZ2wgPQogICAgICAgICAgICBjYW52 YXMuZ2V0Q29udGV4dCgnd2ViZ2wyJyk7CgoKICAgICAgICBjb25zdCBzYWIgPSBuZXcgU2hhcmVk QXJyYXlCdWZmZXIoMTAyNCwgewogICAgICAgICAgICBtYXhCeXRlTGVuZ3RoOiAyMDQ4CiAgICAg ICAgfSk7CgogICAgICAgIGNvbnN0IGRhdGEgPSBuZXcgVWludDhBcnJheShzYWIsIDAsIDMyKTsK ICAgICAgICBsZXQgYnVmZmVyID0gZ2wuY3JlYXRlQnVmZmVyKCk7CiAgICAgICAgZ2wuYmluZEJ1 ZmZlcihnbC5BUlJBWV9CVUZGRVIsIGJ1ZmZlcik7CiAgICAgICAgZ2wuYnVmZmVyRGF0YShnbC5B UlJBWV9CVUZGRVIsIGRhdGEsIGdsLlNUQVRJQ19EUkFXKTsKCiAgICA8L3NjcmlwdD4KPC9ib2R5 PgoKPC9odG1sPg== 464416 2023-01-09 01:43:46 -0800 2023-01-09 01:43:46 -0800 server to run the file with http-server-same-origin.py text/x-python-script 415 kkinnunen IyEvdXNyL2Jpbi9lbnYgcHl0aG9uCmZyb20gaHR0cCBpbXBvcnQgc2VydmVyCgpjbGFzcyBNeUhU VFBSZXF1ZXN0SGFuZGxlcihzZXJ2ZXIuU2ltcGxlSFRUUFJlcXVlc3RIYW5kbGVyKToKICAgIGRl ZiBlbmRfaGVhZGVycyhzZWxmKToKICAgICAgICBzZWxmLnNlbmRfaGVhZGVyKCJDcm9zcy1Pcmln aW4tT3BlbmVyLVBvbGljeSIsICJzYW1lLW9yaWdpbiIpCiAgICAgICAgc2VsZi5zZW5kX2hlYWRl cigiQ3Jvc3MtT3JpZ2luLUVtYmVkZGVyLVBvbGljeSIsICJyZXF1aXJlLWNvcnAiKQogICAgICAg IHNlcnZlci5TaW1wbGVIVFRQUmVxdWVzdEhhbmRsZXIuZW5kX2hlYWRlcnMoc2VsZikKCmlmIF9f bmFtZV9fID09ICdfX21haW5fXyc6CiAgICBzZXJ2ZXIudGVzdChIYW5kbGVyQ2xhc3M9TXlIVFRQ UmVxdWVzdEhhbmRsZXIpCg==