Re: New editor's draft (v20130603)

On 2013-06-04 16:34, Randell Jesup wrote:
> On 6/3/2013 10:37 AM, Adam Bergkvist wrote:
>> Hi
>> A new version of the editor's draft is available.
>> Dated version:
>> Living document:
>> Changes include:
>>     Added IceRestart constraint.
>>     Big updates on DataChannel API to use new channel setup procedures.
>> Please review and provide feedback.
>> Adam (for the editors)
> One issue:   In RTCDataChannelInit:
>      boolean          negotiated  <>  =true;
> The default should be false - it should be set to true on any
> externally-negotiated channel.

Oh, I interpreted preNegotiated (skipped the "pre" -> negotiated) as a 
property on the channel that makes the user agent negotiate the channel 
for you in-band. I.e., it's pre-negotiated (or just negotiated) meaning 
that you (the developer) doesn't have to worry about negotiation; the 
opposite of "externally-negotiated".

Should we use externallyNegotiated (default false) instead? That makes 
sense on both sides.


> We just landed code in FF to implement the rest of this (but with false
> as the default) for Firefox 22 (the first release with RTCPeerConnection
> enabled by default), with the old names as synonyms for the next release
> or so (on Firefox 24 nightlies it will Warn in the Web Console if you
> use the old names).  As of Firefox 24, we've begin to move
> PeerConnection/etc from xpidl to webidl internally, which makes syncing
> with the spec easier.
> We have not yet implemented all of the RTCDataChannel properties; we did
> implement 'id' with the old 'stream' as a temporary synonym, and ordered
> (and a few others) were already supported.
> Thanks for the update!  I'll check the rest of it more closely; sorry I
> didn't get to it before it was published.
> --
> Randell Jesup

Received on Wednesday, 5 June 2013 12:00:08 UTC