Re: RTCSessionDescription and RTCIceCandidate

On 18/09/15 10:06, Dominique Hazael-Massieux wrote:
> On 15/09/2015 20:36, Jan-Ivar Bruaroey wrote:
>> On 9/15/15 12:54 AM, Martin Thomson wrote:
>>> On 14 September 2015 at 20:05, Jan-Ivar Bruaroey <jib@mozilla.com> wrote:
>>>>     -  Promise<void> addIceCandidate (RTCIceCandidate candidate);
>>>>     +  Promise<void> addIceCandidate (optional RTCIceCandidateInit
>>>> candidate);
>>> Why optional, out of interest?
>>
>>    0:18.18 WebIDL.WebIDLError: error: Dictionary argument or union
>> argument containing a dictionary not followed by a required argument
>> must be optional,
>
> Rather than make the argument optional (which is awkward in this case),

Agree.

> we would make the "DOMString candidate" member of RTCIceCandidateInit
> required

It already is IIUC.

> (which removes the obligation of having dictionary arguments be
> optional).
>
> Dom
>
>


Received on Wednesday, 23 September 2015 09:06:20 UTC