-
Notifications
You must be signed in to change notification settings - Fork 166
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
Have a single AudioWorkletGlobalScope
per BaseAudioContext
#1753
Comments
Yes, this is an oversight and the fix should be trivial. So you think the following text in the spec is problematic? (From here)
|
Yes, it means you can't really count on the fact that you can read the expandos you set in one |
Also Worklet's Here's the relevant discussion: https://github.com/WebAudio/web-audio-api/pull/1020/files#r88695255 But I think it might need a completely different approach to support the multi-threading without violating many assumptions that are already in place. Hence, SGTM. |
… could exist for a single AudioContext. This fixes WebAudio#1753.
I think this was forgotten when writing a PR for #1511 (no wonder, it was so big).
The resolution seem to be #1511 (comment). It's also a resolution in #966 (comment).
It's rather critical for interop and is what Chrome implements (and we'll implement). Authors need to reliably be able to share data between instances (wave tables, audio buffers, other precomputed data, etc.).
The text was updated successfully, but these errors were encountered: