Re: PR for adding RtpSender.transport, RtpReceiver.transport, RTCDtlsTransport, RTCIceTransport, etc

On Jul 24, 2015, at 14:08, Peter Thatcher <pthatcher@google.com<mailto:pthatcher@google.com>> wrote:

Is everyone OK with the current state of the PR now?  I made all the edits to address feedback that had consensus.  Is there anything else?

[BA] A few comments:

                + "dom-dtlstransport-transport"><code>RTCDtlsTransport.transport</code></dfn>
                + attribute is the underlying transport that is used to send
                + and receive packets. The underlying transport may not be
                + shared between multiple <code>RTCDtlsTransport</code>
                + objects.

[BA] I might say "multiple active RTCDtlsTransport objects" because that is when the multiplexing Issue arises.

                + This event handler, of event handler event
                + type <code><a href=
                + "#event-icenomincatedcandidatepairchange">icenomincatedcandidatepairchange</a></code>,
                + MUST be fired any time the <a href=
                + "#dom-icetransport-state"><code>RTCIceTransport</code>'s
                + selected candidate pair</a> changes.


[BA] s/nomincated/selected/

RTCIceTransport"
                + <dt>readonly attribute RTCIceConnectionState state</dt>

[BA] Should this be RTCIceTransportState?

Received on Friday, 24 July 2015 12:52:32 UTC