Skip to content

Conversation

@Prospero23
Copy link
Contributor

Revives #5535 and splits it into two commits with fixes for changes requested by @gmta.

Message of original PR:

Summary

Implements the WebAudio control message queue as specified in the Web Audio API

Changes

-Added ControlMessageQueue class with thread-safe operations
-Integrated queue into BaseAudioContext
-Updated AudioContext methods (suspend/resume/close) to use the queue
-Updated AudioScheduledSourceNode to queue start/stop messages

Testing

  • Added custom integration test for the control message queue

Changes lambdas in `OfflineAudioContext` and `AudioContext` to
calculate realm using `this->realm()`.
@Prospero23
Copy link
Contributor Author

I am not sure if WEB_PLATFORM_OBJECT and GC_DECLARE_ALLOCATOR should be on the queue but they were on the original PR so I left them.

Should something that is not declared to JS still be a WEB_PLATFORM_OBJECT? Almost certainly just confusion caused by lack of understanding on my part.

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