RTCIceCandidate: Add missing attributes
Reported by
acmesqua...@gmail.com,
Jan 20 2017
|
||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0 Steps to reproduce the problem: RTCIceCandidate is missing attributes from the spec. What is the expected behavior? https://www.w3.org/TR/webrtc/#rtcicecandidate-interface What went wrong? Not implemented Did this work before? No Does this work in other browsers? Yes Chrome version: 57.0.2986.0 dev (64-bit) Channel: dev OS Version: Flash Version:
,
Jan 23 2017
hbos@: can you take a look at this?
,
Feb 15 2017
hbos@ Gentle Ping! Could you please let us know is there any latest update available on this issue? Thanks!
,
Feb 20 2017
Sorry for the late response. I have not had any time to work on this at all. Does this need to be prioritized?
,
Feb 20 2017
Those exist -
readonly attribute DOMString candidate;
readonly attribute DOMString? sdpMid;
readonly attribute unsigned short? sdpMLineIndex;
Those are missing -
readonly attribute DOMString foundation;
readonly attribute unsigned long priority;
readonly attribute DOMString ip;
readonly attribute RTCIceProtocol protocol;
readonly attribute unsigned short port;
readonly attribute RTCIceCandidateType type;
readonly attribute RTCIceTcpCandidateType? tcpType;
readonly attribute DOMString? relatedAddress;
readonly attribute unsigned short? relatedPort;
I have not tested the serializer, but Chrome exposes a toJSON method.
,
Feb 21 2017
As this issue can't be triaged from TE end removed the Needs-Triage-M57 and added appropriate milestone for this.
,
Mar 22 2017
,
Mar 23 2017
Published spec now also includes ufrag
,
Apr 24 2017
Bumping to M60. Please correct if that's wrong.
,
May 11 2017
deadbeef@, you added these to the spec in https://github.com/w3c/webrtc-pc/pull/325, do you still think it's a good idea to implement this? Nobody else has AFAICT, so just reverting it from the spec would be a quick fix to alignment :)
,
May 11 2017
I think it's a good idea, yes. All of the necessary fields should already be there at the webrtc level.
,
May 15 2017
It makes sense to address this after we've sorted out RTCRtpSender and friends, since that exposes candidates in a new way and will require fixing candidate-related things in the webrtc layer. RTCRtpSender -> RTCDtlsTransport -> RTCIceTransport -> RTCIceCandidate.
,
Mar 7 2018
,
Mar 8 2018
,
Jan 10
Per #11 this should already exist in the WebRTC layer.
,
Jan 10
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by ajha@chromium.org
, Jan 23 2017