Rename RTCRTPStreamStats to match the spec RTCRtpStreamStats

RESOLVED FIXED in Firefox 63

Status

()

P2
normal
RESOLVED FIXED
3 months ago
2 months ago

People

(Reporter: ng, Assigned: ng)

Tracking

({dev-doc-complete})

unspecified
mozilla63
dev-doc-complete
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

3 months ago
This dictionary name has been changed[0] in the spec[1] fixing a deviation from the standard nomenclature. We should update our WebIDL and our docs[2].
[0] https://github.com/w3c/webrtc-stats/pull/359
[1] https://w3c.github.io/webrtc-stats/#dom-rtcrtpstreamstats
[2] https://developer.mozilla.org/en-US/docs/Web/API/RTCStatsReport#RTCRTPStreamStats
(Assignee)

Comment 2

3 months ago
Created attachment 8997174 [details]
Bug 1480498 - P1 - Rename RTCRTPStreamStats in WebIDL to match the spec IDL

This renames the RTCRTPStreamStats partial dictionary to match the recent spec changes
(Assignee)

Comment 3

3 months ago
Created attachment 8997176 [details]
Bug 1480498 - P2 - renaming RTCRTPStreamStats in the WebRTCGlobals

Renaming RTCRTPStreamStats in the WebRTCGlobals.

Updated

3 months ago
Attachment #8997174 - Attachment description: Bug 1480498 - Rename RTCRTPStreamStats in WebIDL to match the spec IDL → Bug 1480498 - P1 - Rename RTCRTPStreamStats in WebIDL to match the spec IDL
Comment on attachment 8997174 [details]
Bug 1480498 - P1 - Rename RTCRTPStreamStats in WebIDL to match the spec IDL

Olli Pettay [:smaug] has approved the revision.

https://phabricator.services.mozilla.com/D2682
Attachment #8997174 - Flags: review+
Comment on attachment 8997176 [details]
Bug 1480498 - P2 - renaming RTCRTPStreamStats in the WebRTCGlobals

Michael Froman [:mjf] has approved the revision.

https://phabricator.services.mozilla.com/D2683
Attachment #8997176 - Flags: review+

Updated

3 months ago
Priority: -- → P2

Comment 6

3 months ago
Pushed by na-g@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/6f781af29708
P1 - Rename RTCRTPStreamStats in WebIDL to match the spec IDL r=smaug

Comment 7

3 months ago
Pushed by na-g@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/308ac9e24b44
P2 - renaming RTCRTPStreamStats in the WebRTCGlobals r=mjf

Comment 8

3 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6f781af29708
https://hg.mozilla.org/mozilla-central/rev/308ac9e24b44
Status: NEW → RESOLVED
Last Resolved: 3 months ago
status-firefox63: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Docs updated:

https://developer.mozilla.org/en-US/docs/Web/API/RTCStatsReport

And Firefox 63 for developers.

We don't currently have any other documentation for this although I hope to soon.

I am working on the BCD update but need to deal with some other stuff before I finish that, so I will not mark this doc complete yet.
BCD work is done:

https://github.com/mdn/browser-compat-data/pull/2802#issuecomment-421009601
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.