Section 3: ICE Transport

[aboba]

From Shijun Sun:

Section 3.3. IceTransport. restart(). If this is to reset to gathering, the spec should be clear on the impact to “state”, local candidate list, etc. And, should we allow any change to the getherPolicy when restart()? And, from another perspective, can we recommend just constructing a new IceTransport as in Section 3.1, i.e. removing “restart()” from the interface?

Section 3.3. IceTransport. The icelistener should not be optional. We can’t have an empty constructor to create the iceTransport.

If gatherpolicy is not valid, gather() will throw invalid access exception.

Call gather() during gathering, throw invalid access exception.

Call gather() after gather-complete. Should UA invalidate all the local candidates reported already? Propose to throw invalid access exception unless the stop() is called explicitly.

Call start() with invalid params will throw invalid access exception

IceTransport. When “error” should be fired? What we’d like to propose: If the transport is closed without either party calling stop(), the UA should fire an error event.

[aboba]

Since there is no way to set the RTCIceListener after construction, I would agree that the RTCIceLIstener cannot be optional in the constructor. Should an InvalidParameter exception be thrown if invalid parameters are passed? Other suggestions seem reasonable.

[robin-raymond]

RTCIceListener IS optional BUT you have to pass in RTCIceOptions in that case.

[robin-raymond]

"onerror" is for things like TURN credentials are invalid? But that should be on the RTCIceListener not the RTCIceTransport. Not sure what other "onerror" s can happen...