Closed
Bug 1347070
Opened 7 years ago
Closed 5 years ago
Add qpSum to local outbound RTCRTPStreamStats
Categories
(Core :: WebRTC, enhancement, P3)
Core
WebRTC
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: ng, Assigned: ng)
References
(Blocks 3 open bugs)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
qpSum [1] is missing from our RTCRTPStreamStats implementation, and need to be added.
Assignee | ||
Comment 1•7 years ago
|
||
^ [1] http://rawgit.com/w3c/webrtc-stats/master/webrtc-stats.html#dom-rtcrtpstreamstats-qpsum
Assignee | ||
Comment 2•7 years ago
|
||
This is available in head on webrtc.org in webrtc/video_receive_stream.h.
Updated•7 years ago
|
Keywords: dev-doc-needed
Comment 3•7 years ago
|
||
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Assignee | ||
Comment 4•5 years ago
|
||
This adds qpSum to RTC RTP stats
Assignee | ||
Comment 5•5 years ago
|
||
As noted in 1519590, changes would need to be made to the WebRtcMediaDataDecoderCodec to support qpSum on inbound.
Summary: Add qpSum to RTCRTPStreamStats → Add qpSum to local outbound RTCRTPStreamStats
Updated•5 years ago
|
Attachment #9035251 -
Attachment description: Bug 1347070 - add qpSum to RTC RTP stats → Bug 1347070 - add qpSum to RTC outbound RTP stats
Assignee | ||
Comment 6•5 years ago
|
||
WebIDL review spec[0] notes:
This patch adds the qpSum[1] field to the RTCRtpStreamStats[2] dictionary.
[0] WebRTC Stats spec: https://w3c.github.io/webrtc-stats/
[1] https://w3c.github.io/webrtc-stats/#dom-rtcrtpstreamstats-qpsum
[2] https://w3c.github.io/webrtc-stats/#streamstats-dict*
Pushed by na-g@nostrum.com: https://hg.mozilla.org/integration/autoland/rev/675a97d54d12 add qpSum to RTC outbound RTP stats r=jib,smaug
Comment 8•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Comment 9•5 years ago
|
||
Note to MDN writers: I've added a note to the Fx66 rel notes: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/66#APIs
Need to check that this is properly documented, and that bcd is up to date, etc.
Comment 10•5 years ago
|
||
The content is already in place. BCD update PR has been submitted:
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•