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

Update configure methods to use a parallel queue to run the the configuration check support #721

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

youennf
Copy link
Contributor

@youennf youennf commented Sep 27, 2023

Make close algorithm synchronously call the error callback and ensure the close algorithm is never called from a parallel queue. Update isConfigSupported to stop resolving the promise from a parallel queue.

This should partially fix #719.


Preview | Diff

…guration check support.

Make close algorithm synchronously call the error callback and ensure the close algorithm is never called from a parallel queue.
Update isConfigSupported to stop resolving the promise from a parallel queue.
@youennf
Copy link
Contributor Author

youennf commented Sep 27, 2023

This PR is changing configure to do the configuration check from the codec work queue.
Assignment of [[codec implementation]] is now done in the codec work queue, while previously it was made from the JS thread. I am not sure why this was written this way. Does somebody know why it was done so?

I also added if needed since we might not always want to create a new [[codec implementation]] each time configure is called (say only bitrate is changing for instance).

@aboba aboba merged commit 3856bbf into w3c:main Sep 28, 2023
16 checks passed
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Sep 28, 2023
SHA: 3856bbf
Reason: push, by aboba

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

The encoder/decoder configure methods is not using a queue to check for configuration support
3 participants