Skip to content

Some AudioWorklet refactoring and improvements#185

Merged
b-ma merged 18 commits into
mainfrom
fix/audioworklet
May 28, 2026
Merged

Some AudioWorklet refactoring and improvements#185
b-ma merged 18 commits into
mainfrom
fix/audioworklet

Conversation

@b-ma

@b-ma b-ma commented May 27, 2026

Copy link
Copy Markdown
Collaborator

WPT Results are now

before

- # pass: 168
- # fail: 21
- # type error issues: 0

after

- # pass: 173
- # fail: 15
- # type error issues: 0

Remaining issues:

  • AudioParam does not behave as expected -> needs to be reviewed in upstream
  • Configurable render quantum size -> to be implemented in upstream
  • Issue with how currentTime and currentFrame are inited -> a bit quite tricky probably, means we should be able to request the information to the Rust audio thread somehow
  • Blob can be deserialized in Node, which makes audioworkletnode-onerror.https.html fail
  • Transfer of input / output underlying arrayBuffers is forbidden on our side (causes Segmentation Fault) -> really don't see why it is allowed anyway, probably won't fix
  • One JS tick per quantum, cf. Yield once to VM per quantum, another thread prio crate, some timings, use condvar instead of crossbeam; no improvements #128
  • Plus one or two minor things

@b-ma b-ma marked this pull request as draft May 28, 2026 04:49
@b-ma b-ma marked this pull request as ready for review May 28, 2026 08:48
@b-ma

b-ma commented May 28, 2026

Copy link
Copy Markdown
Collaborator Author

WPT against upstream main

RESULTS:
- # pass: 5137
- # fail: 440
- # type error issues: 0
> wpt duration: 2:44.291 (m:ss.mmm)

@b-ma b-ma merged commit b5a2529 into main May 28, 2026
1 check passed
@b-ma b-ma deleted the fix/audioworklet branch May 28, 2026 09:49
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