Skip to content
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

[HTML5] Use browser mix rate by default on the Web. #52695

Merged
merged 1 commit into from
Sep 15, 2021

Conversation

Faless
Copy link
Collaborator

@Faless Faless commented Sep 15, 2021

Browsers doesn't really like forcing the mix rate, e.g. Firefox does not allow input (microphone) if the mix rate is not the default one, Chrom* will exhibit worse performances, etc.

Browsers doesn't really like forcing the mix rate, e.g. Firefox does not
allow input (microphone) if the mix rate is not the default one, Chrom*
will exhibit worse performances, etc.
@Faless Faless marked this pull request as draft September 15, 2021 10:14
@Faless
Copy link
Collaborator Author

Faless commented Sep 15, 2021

Sorry, I've converted this to draft because I realized I haven't tested what happens with the dummy driver on the Web (e.g. when the audiocontext fails to start). I fear we might end up with a bunch of division by 0.

@Faless Faless marked this pull request as ready for review September 15, 2021 18:23
@Faless
Copy link
Collaborator Author

Faless commented Sep 15, 2021

According to the discussion in the contributor chat the audio server should still behave correctly with a mix rate of 0, and in my tests, it all worked fine. I'm going to merge, and we'll fix any problem that might comes up.

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.

2 participants