Skip to content

Fix incorrect assumptions in drawingbuffer-storage-test.#3770

Merged
kenrussell merged 1 commit intoKhronosGroup:mainfrom
kenrussell:drawingbuffer-assumptions
Apr 14, 2026
Merged

Fix incorrect assumptions in drawingbuffer-storage-test.#3770
kenrussell merged 1 commit intoKhronosGroup:mainfrom
kenrussell:drawingbuffer-assumptions

Conversation

@kenrussell
Copy link
Copy Markdown
Member

It is not guaranteed that WebGL implementations can allocate a drawing buffer at any specific size. Avoid asserting that allocations up to the maximum renderbuffer size will succeed.

Associated with https://crbug.com/332743717 .

It is not guaranteed that WebGL implementations can allocate a drawing
buffer at any specific size. Avoid asserting that allocations up to
the maximum renderbuffer size will succeed.

Associated with https://crbug.com/332743717 .
@kenrussell kenrussell requested a review from lexaknyazev April 6, 2026 19:21
Copy link
Copy Markdown
Member

@toji toji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@kenrussell kenrussell merged commit 5a01424 into KhronosGroup:main Apr 14, 2026
2 checks passed
@kenrussell kenrussell deleted the drawingbuffer-assumptions branch April 14, 2026 03:51
@tidoust
Copy link
Copy Markdown

tidoust commented Apr 15, 2026

That's probably triggered by something else, but this seems to have broken the WebGL extensions registry. For example, this used to return an actual spec and currently yields a directory structure:
https://registry.khronos.org/webgl/extensions/EXT_frag_depth/

Last modified time seems to match the time at which this PR was merged.

Same problem for all extensions under https://registry.khronos.org/webgl/extensions/

@kenrussell
Copy link
Copy Markdown
Member Author

@tidoust I couldn't remember where I saw this report and just re-found your comment here :) The WebGL extension registry has been fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants