search

sam-vi / webrtc-icecontroller

RTCIceController interface extension to the WebRTC 1.0 Web API
https://sam-vi.github.io/webrtc-icecontroller/
Other
1 stars 0 forks source link

Worth mentioning this replaces the need for continualGatheringPolicy #5

Closed danjenkins closed 1 year ago

danjenkins commented 1 year ago

I don't know if the spec is the right place but the first thing I thought when seeing this on the WebRTC Working group mailing list was whether this would remove the need for "continualGatheringPolicy" which isn't even made publicly available in the browser. From what I've read, this would mean not having to enable that flag to be toggle-able from the browser as you'd have complete control over the ice controller over the course of a webrtc session?

sam-vi commented 1 year ago

I wouldn't say that this proposal removes the need for continualGatheringPolicy. But the use of an ICE controller will certainly benefit from both continual gathering and Trickle ICE.

AFAIK the gathering behaviour is not currently standardized in the Web API spec or any ICE RFCs. So continual gathering cannot be expected to be supported in every browser implementation. As proposed, it would be implementation detail whether, for instance in chromium, continualGatheringPolicy is set to GATHER_CONTINUALLY when an RTCIceController is supplied in the RTCConfiguration.

https://github.com/w3c/webrtc-extensions/issues/121 is the correct forum to pursue standardization of (or some consensus on the support for) gathering policy and expose that as a configuration in the Web API.

danjenkins commented 1 year ago

Thanks @sam-vi ! Appreciate the response!