Closed Bug 1347070 Opened 7 years ago Closed 5 years ago

Add qpSum to local outbound RTCRTPStreamStats

Categories

(Core :: WebRTC, enhancement, P3)

enhancement

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.
Blocks: 1359872
Depends on: 1376873
This is available in head on webrtc.org in webrtc/video_receive_stream.h.
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
This adds qpSum to RTC RTP stats

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
Attachment #9035251 - Attachment description: Bug 1347070 - add qpSum to RTC RTP stats → Bug 1347070 - add qpSum to RTC outbound RTP stats

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
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66

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.

The content is already in place. BCD update PR has been submitted:

https://github.com/mdn/browser-compat-data/pull/3591

You need to log in before you can comment on or make changes to this bug.