Re: [webrtc-encoded-transform] Add presentationTimestamp to RTCEncodedVideoFrameMetadata (#173)

How about us using `timestamp` here inside RTCEncodedVideoFrameMetadata for this presentation timestamp field, to match WebCodecs, and I'll create a follow-up PR to also add `rtpTimestamp` inside this Metadata and deprecating/removing the `RTCEncodedVideoFrame.timestamp` field?

This moves us into more alignment with the WebCodecs type by having a consistent `timestamp` field, gets away from the confusion of having two `timestamp`s without needing to have an immediate breaking change, and preserves the amount of information available to web apps.

-- 
GitHub Notification of comment by tonyherre
Please view or discuss this issue at https://github.com/w3c/webrtc-encoded-transform/pull/173#issuecomment-1514522422 using your GitHub account


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

Received on Wednesday, 19 April 2023 10:48:39 UTC