Closed Bug 1489040 Opened 2 years ago Closed 2 years ago

WebRTC ICE candidate stats ipAddress needs to be renamed

Categories

(Core :: WebRTC, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: ng, Assigned: ng)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete, site-compat)

Attachments

(1 file)

It looks like the spec [0] is moving towards[1] the name 'address' instead of 'ip'. Either way things shake out we will need to rename 'ipAddress'[2].

[0] https://w3c.github.io/webrtc-stats/#dom-rtcicecandidatestats-ip
[1] https://github.com/w3c/webrtc-pc/issues/1913 *
[2] https://searchfox.org/mozilla-central/source/dom/webidl/RTCStatsReport.webidl#156

*Note the spec issue is filed against the webrtc-pc not webrtc-stats spec, but mentions the need to mirror the change in the webrtc-stats spec.
Status: NEW → ASSIGNED
Rank: 25
Priority: -- → P3
Bug 1489040 - P1 - Update WebRTC ICE candidate stats field ipAddress to new name, address
Bug 1489040 - P2 - ajust WebRTC stats mochitest for new stat name 'address'
Bug 1489040 - P3 - add test for legacy WebRTC stat names
Comment on attachment 9008126 [details]
Bug 1489040 Update WebRTC ICE candidate stats field name ipAddress to address

Jan-Ivar Bruaroey [:jib] (needinfo? me) has approved the revision.
Attachment #9008126 - Flags: review+
Comment on attachment 9008126 [details]
Bug 1489040 Update WebRTC ICE candidate stats field name ipAddress to address

Michael Froman [:mjf] has approved the revision.
Attachment #9008126 - Flags: review+
Pushed by na-g@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/68d20ffd470d
Update WebRTC ICE candidate stats field name ipAddress to address r=mjf,jib,smaug
https://hg.mozilla.org/mozilla-central/rev/68d20ffd470d
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Note to MDN writers:

This is covered in the rel notes under the general note about 
RTCIceCandidateStats; see
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/65#APIs

I'm setting this to dev-doc-complete, because the other related bug (1324788) will cover this when the documentation is done.
You need to log in before you can comment on or make changes to this bug.