Closed
Bug 1480498
Opened 5 years ago
Closed 5 years ago
Rename RTCRTPStreamStats to match the spec RTCRtpStreamStats
Categories
(Core :: WebRTC, enhancement, P2)
Core
WebRTC
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: ng, Assigned: ng)
Details
(Keywords: dev-doc-complete)
Attachments
(2 files)
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 1•5 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=64d452f8ae9f303f7c04834cd08fcbb8fac7fc99
Assignee | ||
Comment 2•5 years ago
|
||
This renames the RTCRTPStreamStats partial dictionary to match the recent spec changes
Assignee | ||
Comment 3•5 years ago
|
||
Renaming RTCRTPStreamStats in the WebRTCGlobals.
Updated•5 years 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 4•5 years ago
|
||
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 5•5 years ago
|
||
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•5 years ago
|
Priority: -- → P2
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
Pushed by na-g@nostrum.com: https://hg.mozilla.org/integration/autoland/rev/308ac9e24b44 P2 - renaming RTCRTPStreamStats in the WebRTCGlobals r=mjf
Comment 8•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6f781af29708 https://hg.mozilla.org/mozilla-central/rev/308ac9e24b44
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Comment 9•5 years ago
|
||
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.
Comment 10•5 years ago
|
||
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.
Description
•