Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chrome 130 supports WebAssemblyCompileOptions, which enable JS String Builtins #25391

Merged
merged 13 commits into from
Dec 16, 2024

Conversation

chrisdavidmills
Copy link
Contributor

Summary

Chrome 130 supports WebAssembly JS String builtins. Specifically, This PR adds data points for the compileOptions parameter used to enable these built-ins, which is added to:

Test results and supporting details

See https://chromestatus.com/feature/6695587390423040 for the data source. I have also tested each of the above cases and found them to work by default in Chrome 130/131.

Related issues

@github-actions github-actions bot added data:wasm Compat data for Web Assembly features. https://developer.mozilla.org/en-US/docs/WebAssembly size:l [PR only] 101-1000 LoC changed labels Dec 12, 2024
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

LGTM overall, just the spec_url doesn't seem specific enough.

PS: It is what we currently do, but I really don't like this data duplication (6 feature for what is likely covered by the same implementation) and would prefer a way to inherit support data from another feature.

webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api/Module.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api.json Outdated Show resolved Hide resolved
webassembly/api/Module.json Outdated Show resolved Hide resolved
@chrisdavidmills
Copy link
Contributor Author

@Elchi3 @caugner cheers gents; updates applied.

@caugner caugner changed the title Add data points for compileOptions, which enable wasm JS string builtins Chrome 130 supports WebAssemblyCompileOptions, which enable JS String Builtins Dec 16, 2024
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@caugner caugner merged commit bd889a2 into mdn:main Dec 16, 2024
9 checks passed
@mdn-bot mdn-bot mentioned this pull request Dec 16, 2024
@chrisdavidmills chrisdavidmills deleted the wasm-js-string-builtins branch December 16, 2024 12:19
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Jan 18, 2025
Reading across the following links, it appears that Firefox implemented the same thing as Chrome, but when Chris added the `compile_options` keys they weren't reconciled with the existing Firefox data.

- https://bugzilla.mozilla.org/show_bug.cgi?id=1863794
- mdn#25230
- mdn#25391

Discovered in the course of web-platform-dx/web-features#2549
Elchi3 pushed a commit that referenced this pull request Jan 20, 2025
…5706)

Show Firefox support across all of wasm JS string built-ins

Reading across the following links, it appears that Firefox implemented the same thing as Chrome, but when Chris added the `compile_options` keys they weren't reconciled with the existing Firefox data.

- https://bugzilla.mozilla.org/show_bug.cgi?id=1863794
- #25230
- #25391

Discovered in the course of web-platform-dx/web-features#2549
ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:wasm Compat data for Web Assembly features. https://developer.mozilla.org/en-US/docs/WebAssembly size:l [PR only] 101-1000 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants