Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add capture/receive/RTP timestamps definitions for RTCRtpReceiver frames #233

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

guidou
Copy link

@guidou guidou commented Mar 20, 2025

  • add WebRTC-specific interactions with capture/receive/RTP timestamps
  • Rebase and update
  • Fix some style stuff & undo webrtc-extensions.js changes.
  • Address reviewer comments. Use SHOULD/MUST instead of is. Remove informative as things should be observable.
  • Offline review with henbos@: removed ambiguous text.
  • Address review changes

Preview | Diff

@guidou guidou changed the title timestamps Add capture/receive/RTP timestamps definitions for RTCRtpReceiver frames Mar 20, 2025
as specified in [[?RFC3550]] Section 6.4.1, or by other alternative means if use by RTCP SR
isn't feasible.
The capture time of an incoming frame is available if its constituent RTP packets contain the
[=RTP Header Extension for Absolute Capture Time=].
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When the relevant information is available (RTCP SR or header extension) then I think the UA "MUST" provide an estimate, and if not it MUST NOT provde an estimate

Copy link
Member

@jan-ivar jan-ivar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM with change from @henbos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants