-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
FF115 RTCRtpReceiver.jitterBufferTarget supported #33198
Conversation
The actual user agent jitter buffer target will vary between maximum and minimum allowed values that reflects a target range that the user agent can provide based on network conditions and memory constraints, and can change at any time. | ||
The value returned by `jitterBufferTarget` is not affected by the actual target of the user agent. | ||
|
||
The change in average delay can be gradually observed over time by measuring the delta [`RTCInboundRtpStreamStats.jitterBufferDelay`](/en-US/docs/Web/API/RTCInboundRtpStreamStats) divided by the delta [`RTCInboundRtpStreamStats.jitterBufferEmittedCount`](/en-US/docs/Web/API/RTCInboundRtpStreamStats). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note, these two stats are not documented yet, so I have chosen to link to the parent doc.
Preview URLs
Flaws (1)Note! 2 documents with no flaws that don't need to be listed. 🎉 URL:
External URLs (1)URL:
(comment last updated: 2024-04-26 01:56:19) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. One comment for you to look at regarding use of "delta" for calculating average delay (vs. change in average delay?) but leaving a +1 👍🏻
Thanks for the review @bsmth - I modified the part you were concerned about (#33198 (comment)) and I'm happy it addresses your comment, so merging. |
Looks good to me, let's merge 👍🏻 |
FF115 added support for
RTCRtpReceiver.jitterBufferTarget
in https://bugzilla.mozilla.org/show_bug.cgi?id=1592988. This was caught by BCD collector and added to BCD in mdn/browser-compat-data#22871This adds docs for the property and an FF release note update.