Re: [webrtc-pc] Adding note about when ICE restarts are recommended.

Even with Firefox's implementation, I'd argue that doing an ICE restart on "disconnected" is far better than doing it on "failed" or doing nothing. And even if it ends up being pointless, it shouldn't have a *negative* effect besides wasting some bandwidth.

I tried to allude to your points:

> Performing an ICE restart is recommended when iceConnectionState transitions to "disconnected", which **may occur due to a flaky network** but **may** also indicate an issue requiring an ICE restart to repair. **Using other sources of information (such as from getStats) could help make a more informed decision.**

Is there alternative wording you'd suggest? Replace "is recommended" with "may be desired" or something less strong? I'm completely open to suggestions, I just want to say *something* about this.

-- 
GitHub Notification of comment by taylor-b
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/pull/1910#issuecomment-398910898 using your GitHub account

Received on Wednesday, 20 June 2018 21:57:36 UTC