Re: [webrtc-pc] Add detailed steps for constructing RTCIceCandidate

Rebased. I have added reference to `candidate-attribute` for the new `component` field in `RTCIceCandidate`. I also clarify the parsing step to require the parsing to either all pass or fail. e.g. If `component-id` has a value other than `1` or `2`, or if the IP address is malformed, the steps are aborted and all derivative attributes are set to null even if the other field values are valid. This is to prevent browsers from doing non strict parsing which could make it difficult to interpret the fields.

Please let me know if the proposal is sound. Frankly I feel that the simpler approach is to make all attributes non-nullable and simply throw an error when parsing failed. Because otherwise it would be hard for users to use these fields if their values are nullable depending on whether parsing is successful.

-- 
GitHub Notification of comment by soareschen
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/pull/1229#issuecomment-304570249 using your GitHub account

Received on Monday, 29 May 2017 04:32:44 UTC