Elad Alon
Elad Alon
> For video, MediaStreamTrack stats is hopefully sufficient to detect these malfunctioning cases. > For audio, it is unclear whether MediaStreamTrack stats is enough, maybe this should get fixed. When...
Note that this is currently under `produceCropTarget()`. Depending on which PR is merged first - this one or #50 - the other one will have to be updated.
During today's Editors' meeting, @jan-ivar and @youennf have been opposed to merging this PR. If you've changed your mind - great, let's merge. Otherwise - could you please explain why?...
What's the explanation for not merging this PR, btw? Is it the same as #54, or is it different? I've found this side, which someone is intending to present at...
@jan-ivar and @youennf, we have previously discussed how allowing cropTo() to fail was uncontroversial (as opposed to token-minting). Let's have the spec reflect that.
1. I've made the suggested changes. PTAL. 2. During today's Editors' meeting, @jan-ivar and @youennf have been opposed to merging this PR. If you've changed your mind - great, let's...
> 1. This PR needs more work to be merged. I made for instance the comment that the new check should be done asynchronously. @jan-ivar mentioned a different error type...
> Should we reconsider our position and directly use string identifiers? I think that'd be for the best. Modulo the next points. > In that case, should we go with...
**Practically speaking**, I think `CropTarget` with serialization and a possible future-path to stringification are good enough, and I am fine with sticking with them, as the consensus appears to be...