Skip to content

Increase stack size in test browser.test_audioworklet_worker to 4KB#26837

Merged
juj merged 1 commit intoemscripten-core:mainfrom
juj:increase_test_audioworklet_worket_stack_size
May 2, 2026
Merged

Increase stack size in test browser.test_audioworklet_worker to 4KB#26837
juj merged 1 commit intoemscripten-core:mainfrom
juj:increase_test_audioworklet_worket_stack_size

Conversation

@juj
Copy link
Copy Markdown
Collaborator

@juj juj commented May 1, 2026

Increase stack size in test browser.test_audioworklet_worker to 4KB, to fit emscripten_outf() call in the Worker.

@juj juj enabled auto-merge (squash) May 1, 2026 20:30
@juj
Copy link
Copy Markdown
Collaborator Author

juj commented May 2, 2026

This was already approved in review as part of #26835 so I'll land this in.

@juj juj disabled auto-merge May 2, 2026 07:12
@juj juj merged commit abf9a60 into emscripten-core:main May 2, 2026
30 checks passed
@juj
Copy link
Copy Markdown
Collaborator Author

juj commented May 2, 2026

After fixing the problem #26823 with this PR, the test began to work correctly in Firefox, but surprisingly, it now fails in Chrome.

When CircleCI ran the browser tests in Chrome as part of this PR, the suite came out green:

image

But (TIL) there is another CircleCI task that runs on the main branch post-landing. And that is now red:

image

(failing on browser.test_audioworklet_worker test)

There is a race condition problem in the test. I posted PR #26839 to address that. I see that test is now green on CircleCI, so maybe that fixes it.

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.

1 participant