- From: Justin Uberti <juberti@google.com>
- Date: Wed, 12 Jun 2013 15:26:35 -0700
- To: Jan-Ivar Bruaroey <jib@mozilla.com>
- Cc: "public-webrtc@w3.org" <public-webrtc@w3.org>
- Message-ID: <CAOJ7v-0eChtsOs1kDEZy64=OdWuKrXiddU5c2Fm8N8GTxbvzcA@mail.gmail.com>
Works for me. On Wed, Jun 12, 2013 at 12:48 PM, Jan-Ivar Bruaroey <jib@mozilla.com> wrote: > 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: > > dictionary RTCIceServer { 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: > > dictionary RTCIceServer { 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: > > dictionary RTCIceServer { 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 type DOMString, 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. > > > >
Received on Wednesday, 12 June 2013 22:27:21 UTC