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

> Why is `presentationTimestamp` defined with a different type than `presentationTime` as defined in [VideoFrameCallbackMetadata](https://wicg.github.io/video-rvfc/#video-frame-metadata-callback)? If these are defining the same thing, can we be consistent?

The primary motivator is for being able to match with VideoFrame.timestamp, which uses a long long timestamp. I was trying to also pull in the rVFC field, per the discussion towards the end of https://github.com/w3c/webrtc-encoded-transform/pull/137, but that was probably a mistake. Per the comment thread with Jan-Ivar and Youenn, it seems to make sense to define this only in reference with the VideoFrame timestamp - I've pushed a commit removing the ref from the text at least. Does that make sense to you @aboba?

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


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

Received on Wednesday, 12 April 2023 12:50:46 UTC