Re: [webrtc-tests] webrtc: require remote video to be loaded (#4114)

Hm. The events are generated straight from the other PC's onicecandidate handler, which means that it's the implementation under test that's generating whatever is there (null or undefined).
If it's not generating null when it should, the test should fail.
And if it's not clear from the spec what happens when you do addIceCandidate(undefined), then that's a spec bug :-)
Let's make the code simpler and just call addIceCandidate(event.candidate), with a comment that the candidate is supposed to be null for the end-of-candidates.

View on GitHub: https://github.com/w3c/web-platform-tests/pull/4114#issuecomment-257261148

Received on Monday, 31 October 2016 10:34:00 UTC