Re: [webrtc-pc] addIceCandidate end of candidates woes

If we do bad hacks then I don't see a reason not to allow null in addIceCandidate.

I have written code that assumes that for any candidate event where e.candidate is set the candidate can be parsed. This code will probably throw an exception (inside the event handler) when trying to parse an empty string.

You might also want to revert https://github.com/webrtc/samples/pull/866 if you think it is incorrect.

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

Received on Tuesday, 16 May 2017 18:19:19 UTC