Re: [webrtc-pc] RTCRtpCodecCapability is superfluous and should be deleted from the spec. (#2974)

Is the consistency concern over naming or hierarchy?

RTCRtpCodecs appear both in getCapabilities() and getParameters() https://github.com/w3c/webrtc-pc/pull/2834 so it seems wrong to label them either.

They're valid inputs to `setCodecPreferences` and `setParameters(Object.assign(params, {codec}))`

Hierarchy-wise It seems wrong for RTCRtpCodecParameters to inherit from RTCRtpCodecCapability.

IMHO the word "Capability" isn't carrying a lot of weight here.

Would s/RTCRtpHeaderExtensionCapability/RTCRtpHeaderExtension/ help?

-- 
GitHub Notification of comment by jan-ivar
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/issues/2974#issuecomment-2168339881 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 14 June 2024 16:09:55 UTC