Re: Conclusion of RTCIceServer multiple-URI thread

On 6/12/13 11:54 AM, Justin Uberti wrote:
> Summarizing from the thread "Allowing RTCIceServer to contain multiple 
> URLs", here are the recommended changes to be included into the spec. 
> I also noted that the previous agreement on adding the |username| 
> field has not yet made it into the editor's draft, so it is included 
> in the changes below.
>
> Current:
>
> dictionaryRTCIceServer  {
>      DOMString   url  <http://dev.w3.org/2011/webrtc/editor/webrtc.html#widl-RTCIceServer-url>;
>      DOMString?  credential  <http://dev.w3.org/2011/webrtc/editor/webrtc.html#widl-RTCIceServer-credential>;
> };
>
> Proposed:
>
> dictionaryRTCIceServer  {
>      DOMString[]   url  <http://dev.w3.org/2011/webrtc/editor/webrtc.html#widl-RTCIceServer-url>s;  // for backwards compatibility, implementations
>                          // should also handle |url|
Seems good to me, except lets make it:

dictionaryRTCIceServer  {
     sequence<DOMString>   url  <http://dev.w3.org/2011/webrtc/editor/webrtc.html#widl-RTCIceServer-url>s;  // for backwards compatibility, implementations
                                 // should also handle |url|

For the same reason I posted for iceServers: "[Make it] a sequence 
rather than a webidl array, because sequence is passed by value 
(represented in JS by an actual JS array), and [] is passed by reference 
(represented in JS by some weird array-like object, according to our 
webidl guy)".

.: Jan-Ivar :.

>
>     DOMString?   username  <http://dev.w3.org/2011/webrtc/editor/webrtc.html#widl-RTCIceServer-url>;  // previously agreed upon
>      DOMString?  credential  <http://dev.w3.org/2011/webrtc/editor/webrtc.html#widl-RTCIceServer-credential>;
> };
>
>
>           4.2.2.1 Dictionary |RTCIceServer|
>           <http://dev.w3.org/2011/webrtc/editor/webrtc.html#idl-def-RTCIceServer> Members
>
>
>
> ||username|of typeDOMString, nullable |
> ||
> |If the url element of the internal array is a TURN URI, then this is 
> the username to use with that TURN server.|
> |credential| of type DOMString, nullable
>
>     If the url element of the internal array is a TURN URI, then this
>     is the password to use with that TURN server.
>
> |urls| of type DOMString[]
>
>     A list of STUN or TURN URIs as defined in [STUN-URI
>     <http://dev.w3.org/2011/webrtc/editor/webrtc.html#bib-STUN-URI>]
>     and [TURN-URI
>     <http://dev.w3.org/2011/webrtc/editor/webrtc.html#bib-TURN-URI>].
>     If multiple URIs are present, they represent alternate ways to
>     contact the STUN/TURN server, listed in order of preference.
>
>