[[active encoder config]] may be incorrect when running the Output EncodedVideoChunk algorithm.
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 1.3k
- Forks
- 194
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 3
Description
As of #113, The Output EncodedVideoChunk algorithm specifies that parts of the output config are to be copied from the "active encoder config". But the spec doesn't guarantee that the "active encoder config" was actually used to encode the given output (as opposed to some earlier config).
One fix is to specify that videoEncoder.configure() must flush all pending outputs before applying the new configuration. This is a little heavy though - we don't really want to require UAs to flush in this way (even though chromium currently does). There are some scenarios where the pipeline may be fairly deep and the configuration change fairly small such that a flush is not required and inefficient. I think a flush may not actually be observable though (as the running the configure control message is not observable), so maybe it leaves implementers the option to pursue equivalent alternatives.
Another fix is just to write out in english (rather than in algorithm steps) how the emitted VideoDecoderConfig is to be populated.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the Output EncodedVideoChunk algorithm and the videoEncoder.configure() behavior described in this issue, then review the context from #113. Determine whether configuration changes must flush pending outputs or whether the emitted VideoDecoderConfig should instead be specified in prose, and document the selected resolution.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- html
- Domain
- api, web-dev
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100