Closed
Bug 970682
Opened 10 years ago
Closed 10 years ago
Add loss, jitter, and RTT to about:webrtc
Categories
(Core :: WebRTC: Audio/Video, defect)
Core
WebRTC: Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla30
People
(Reporter: abr, Assigned: jib)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
2.85 KB,
patch
|
bwc
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Comment 1•10 years ago
|
||
Once these are added to the stats API, we need to display them on about:webrtc.
Assignee | ||
Comment 2•10 years ago
|
||
Try - https://tbpl.mozilla.org/?tree=Try&rev=f62c29c3ef6e
Attachment #8391431 -
Flags: review?(docfaraday)
Assignee | ||
Comment 3•10 years ago
|
||
Loss and jitter are covered by Bug 970686, so this is the last patch (i.e. no need to leave this open)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jib
Comment 4•10 years ago
|
||
Comment on attachment 8391431 [details] [diff] [review] Add RTT to about:webrtc Review of attachment 8391431 [details] [diff] [review]: ----------------------------------------------------------------- Looks pretty much good, with just a minor concern. ::: toolkit/content/aboutWebrtc.xhtml @@ +157,5 @@ > if (stat.bytesReceived !== undefined) { > statsString += " (" + round00(stat.bytesReceived/1024) + " Kb)"; > } > + statsString += " Lost: " + stat.packetsLost + " Jitter: " + stat.jitter; > + if (stat.mozRtt !== undefined) { Is this expected to be undefined in normal operation?
Attachment #8391431 -
Flags: review?(docfaraday) → review+
Assignee | ||
Comment 5•10 years ago
|
||
(In reply to Byron Campen [:bwc] from comment #4) > > + if (stat.mozRtt !== undefined) { > > Is this expected to be undefined in normal operation? Yes, glad you asked. RTT is only included in the RTCP case on the outbound side.
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/449272159e81
Flags: in-testsuite+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/449272159e81
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in
before you can comment on or make changes to this bug.
Description
•