Re: [webrtc-pc] RTCIceCandidate constructor should not require sdpMid/sdpMLineIndex

> My main point in this bug is that no engine currently requires one of sdpMid/sdpMLineIndex to be present, and adding that requirement comes with a compat risk, as any code in the wild that doesn't include one of these will start throwing.

I don't think this matters much, since any code in the wild doing that would fail immediately afterwards inside `addIceCandidate`. But if we want to do all the parameter checking in `addIceCandidate`, I don't see a problem with that; there's less duplication that way.

GitHub Notification of comment by taylor-b
Please view or discuss this issue at using your GitHub account

Received on Friday, 12 May 2017 16:51:31 UTC