Re: [mediacapture-record] Add `audioConstantBitRate` flag to `MediaRecorderOptions`. (#185)

Hi group, [the comment](https://github.com/w3c/mediacapture-record/pull/185#issuecomment-553565231) was from me. I maintain the media mime parsing code in Chrome and I work on the MediaCapabilities spec where mime/codec strings are a key input. 

Both recording and playback operate on codecs in containers. Right now, when it comes to opus at least, we are unified in the the way its described. 

To date, "opus.cbr" is not a valid codec string. All UAs that support opus playback implicitly support both cbr and vbr, but querying any playback API's (canPlayType(), MediaCapabilities, isTypeSupported()) with "opus.cbr" will get a return value that indicates non-support (because they don't recognize it).

The need to signal CBR support is specific to the domain of MediaRecorder, and not necessarily specific to opus. 

Adding an attribute that is external to the codec string solves the issue and avoids the pitfalls above. 



-- 
GitHub Notification of comment by chcunningham
Please view or discuss this issue at https://github.com/w3c/mediacapture-record/pull/185#issuecomment-554525442 using your GitHub account

Received on Friday, 15 November 2019 20:57:36 UTC