search

w3c / webrtc-encoded-transform

WebRTC Encoded Transform
https://w3c.github.io/webrtc-encoded-transform/
Other
122 stars 26 forks source link

Move RTP timestamp members to Metadata objects #204

Closed tonyherre closed 9 months ago

tonyherre commented 10 months ago

Fixes https://github.com/w3c/webrtc-encoded-transform/issues/203

Preview | Diff

tonyherre commented 10 months ago

@alvestrand @fippo explicitly

tonyherre commented 10 months ago

Thanks all!

I wonder what the transition plan is for shipping implementations. Maybe the use is low enough that there is no need to keep both for some amount of time?

My intention for chromium is to have both for some time. We don't have use counters yet though, so I'm not sure how long that would have to be.

fippo commented 10 months ago

Sseveral CPaaS vendors have announced support for client side encryption which means an update cycle of at least six months I'd say.

alvestrand commented 9 months ago

Editors meeting: Needs rebase, can be merged when rebased.

tonyherre commented 9 months ago

Rebased, thanks.

fippo commented 9 months ago

I get validation failures in #140 due to the leftover RTCEncodedVideoFrame/timestamp et al which were removed here but that wasn't picked up by the validator so passed?

saschanaz commented 9 months ago

Just saw this while maintaining https://github.com/microsoft/TypeScript-DOM-lib-generator/, is it safe to remove this without webcompat issue while it's shipped on every engine per MDN?