Re: [Bug 25976] DTMFSender.insertDTMF steps should validate the values of duration and interToneGap.

On 2014-06-09 13:32, Harald Alvestrand wrote:
> On 06/09/2014 10:35 AM, bugzilla@jessica.w3.org wrote:
>> https://www.w3.org/Bugs/Public/show_bug.cgi?id=25976
>>
>> Adam Bergkvist <adam.bergkvist@ericsson.com> changed:
>>
>>             What    |Removed                     |Added
>> ----------------------------------------------------------------------------
>>
>>               Status|NEW                         |RESOLVED
>>           Resolution|---                         |FIXED
>>
>> --- Comment #2 from Adam Bergkvist <adam.bergkvist@ericsson.com> ---
>> Added default values to the idl and updated the algorithm.
>>
>> Proposed fix:
>> https://github.com/fluffy/webrtc-w3c/commit/02f5a3ac13c9c343747be75b8d8a2a874d5b8d8c
>>
>>
>> Closing bug.
>>
> Looks mainly good to me.
>
> Nittish comment:
>
> I think
>
> <dt>void insertDTMF(in DOMString tones, optional long duration = 100,
>   +        long interToneGap = 70)</dt>
>
> should be
>
> <dt>void insertDTMF(in DOMString tones, optional long duration = 100,
>   +        optional long interToneGap = 70)</dt>
>
> As far as I read section 3.2.3 of http://www.w3.org/TR/WebIDL/ you have
> to have the keyword "optional" in front of anything that's declared with
> a default value (grammar rule [44]).

That's right. I'm pretty sure that Respec.js used to automatically add 
optional to the subsequent arguments if one was optional (at one point 
we ended up with double optional when we added them manually).

It's fixed now an I've gone through both docs to check for similar cases.

/Adam

Received on Wednesday, 11 June 2014 06:28:32 UTC