Skip to content

[Gecko Bug 1972621] Change WPT test to specify maximum memory limit. #53260

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

Merged
merged 1 commit into from
Jun 30, 2025

Conversation

moz-wptsync-bot
Copy link
Collaborator

Differential Revision: https://phabricator.services.mozilla.com/D254035

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1972621
gecko-commit: d3b0b2d4a9e6d20457b8b92aba2dbcbee5ae0627
gecko-reviewers: rhunt

Differential Revision: https://phabricator.services.mozilla.com/D254035

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1972621
gecko-commit: d3b0b2d4a9e6d20457b8b92aba2dbcbee5ae0627
gecko-reviewers: rhunt
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Firefox project.

@lutien
Copy link
Contributor

lutien commented Jun 23, 2025

Could somebody from @web-platform-tests/interop take a look at this PR?

@foolip
Copy link
Member

foolip commented Jun 30, 2025

@jakobkummerow can you help review this change for Chrome?

FWIW, the Chrome results are not changed:
https://wpt.fyi/results/wasm/jsapi/memory?diff&filter=ADC&run_id=5126718245568512&run_id=5178732849659904

@jakobkummerow
Copy link

LGTM

Copy link
Member

@foolip foolip left a comment

Choose a reason for hiding this comment

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

Approving on behalf of Chrome.

@moz-wptsync-bot moz-wptsync-bot merged commit a6f8a32 into master Jun 30, 2025
25 checks passed
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1972621 branch June 30, 2025 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants