Re: [webrtc-pc] Candidate from onicecandidate event and addIceCandidate are incompatible

> Does it only return a single event with null to indicate end-of-candidates or does it return both

Both.

> It is: https://github.com/w3c/webrtc-pc/blob/387e0bdb4e7db21e754209f8f0a1bb5e2f3cb333/webrtc.html#L2286

Oh, that appears to just be one place we forgot to update. In the WebIDL it's not nullable.

> And if we really want to keep backwards compatibility to the specification, I think allowing null is correct. 

But, as far as I'm aware, no browser implemented accepting a null argument. So there shouldn't be a backwards compatibility concern.

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

Received on Friday, 17 March 2017 21:36:53 UTC