Re: [webrtc-pc] RTCIceCandidateInit has no required fields, so should probably be optional in addIceCandidate (#2125)

`addIceCandidate({candidate: ""})` is quite weird to me. I like [how the ORTC API has solved this](http://draft.ortc.org/#dom-rtcicecandidatecomplete).

However, I don't have a strong opinion here either and would be fine with keeping it. From a practical perspective, I haven't heard of issues from users and AFAIK browsers haven't even implemented handling this.

-- 
GitHub Notification of comment by lgrahl
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/issues/2125#issuecomment-472211820 using your GitHub account

Received on Tuesday, 12 March 2019 22:53:23 UTC