It seems returning the error break lot of things: https://treeherder.mozilla.org/jobs?repo=try&revision=e85c79108f0dff77506a6f87a0bad8fc494a0949&selectedTaskRun=WU5gMuNmQPuQ56uf8pcr7Q.0
Bug 1839993 Comment 3 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
It seems returning the error break lot of things: https://treeherder.mozilla.org/jobs?repo=try&revision=e85c79108f0dff77506a6f87a0bad8fc494a0949&selectedTaskRun=WU5gMuNmQPuQ56uf8pcr7Q.0. The alternative is to have an parameter indicating if the decode error should be ignored in `Drain(aIgnoreError = true)`. The default value can be `true` so most of the code won't be changed.
It seems returning the error breaks lot of things: https://treeherder.mozilla.org/jobs?repo=try&revision=e85c79108f0dff77506a6f87a0bad8fc494a0949&selectedTaskRun=WU5gMuNmQPuQ56uf8pcr7Q.0. The alternative is to have an parameter indicating if the decode error should be ignored in `Drain(aIgnoreError = true)`. The default value can be `true` so most of the code won't be changed.
It seems returning the error breaks lot of things: https://treeherder.mozilla.org/jobs?repo=try&revision=e85c79108f0dff77506a6f87a0bad8fc494a0949&selectedTaskRun=WU5gMuNmQPuQ56uf8pcr7Q.0. The alternative is to have an parameter indicating if the decode error should be ignored in `Drain(aIgnoreError = true)`. The default value can be `true` so most of the code won't be changed.
It seems returning the error breaks lot of things: https://treeherder.mozilla.org/jobs?repo=try&revision=e85c79108f0dff77506a6f87a0bad8fc494a0949&selectedTaskRun=WU5gMuNmQPuQ56uf8pcr7Q.0. The alternative is to have an parameter indicating if the decode error should be ignored in `Drain(aIgnoreError = true)`. The default value can be `true` so most of the code won't be changed, and we don't set it to `false` in WebCodecs case.