Queueing of candidate events #152

Closed
aboba opened this Issue Sep 27, 2014 · 2 comments

Projects

None yet

2 participants

@aboba
Contributor
aboba commented Sep 27, 2014

Currently the document does not specify what happens between when an RTCIceGatherer object is created, and when an onlocalcandidate event handler is provided. Presumably the RTCIceGatherer begins gathering once created and queues candidate events until an onlocalcandidate event handler is assigned, correct?

@ibc
Contributor
ibc commented Sep 27, 2014

Makes sense. It should buffer all the candidates and fire events when the handler is provided.

@aboba
Contributor
aboba commented Sep 30, 2014

Proposed text for Section 5.3.1 is as follows:

onlocalcandidate of type EventHandler, , nullable
This event handler, of event handler event type icecandidate, must be supported by all objects implementing the RTCIceGatherer interface. It receives events when a new local ICE candidate is available. Since ICE candidate gathering begins once an RTCIceGatherer object is created, candidate events are queued until an onlocalcandidate event handler is assigned.

@aboba aboba closed this Oct 10, 2014
@robin-raymond robin-raymond pushed a commit to robin-raymond/ortc that referenced this issue Oct 14, 2014
Robin Raymond Address questions about RTCDtlsTransport.start(), as noted in: Issue #…
…146

Address questions about RTCRtpCodecCapability.preferredPayloadType, as noted in: Issue #147
Address questions about RTCRtpSender.setTrack() error handling, as noted in: Issue #148
Partially address 'automatic' use of scalable video coding (in RTCRtpReceiver.receive()) as noted in: Issue #149
Renamed RTCIceListener to RTCIceGatherer as noted in: Issue #150
Added text on multiplexing of STUN, TURN, DTLS and RTP/RTCP, as noted in: Issue #151
Address issue with queueing of candidate events within the RTCIceGatherer, as noted in: Issue #152
Clarify behavior of RTCRtpReceiver.getCapabilities(kind), as noted in: Issue #153
7a48e0b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment