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

Giving concrete guidance here is hard since "it depends". Also as Lennart points out a videochat has different requirements from a filetransfer.

I can sign off this:
```
Performing an ICE restart is required when iceConnectionState transitions to "failed".
```
For a recommendation:
```
An application may additionally choose to listen for the iceConnectionState transition to "disconnected" and then use other sources of information (such as using getStats to measure if the number of bytes sent or received over the next couple of seconds increases) to determine whether an ice restart is advisable.
```
So disconnected, getStats, wait 2 seconds, do getStats again (and also check if iceConnectionState is still disconnected!), figure out nothing is sent/received, then do an ice restart.

Thankfully @henbos fixed [this issue](https://bugs.chromium.org/p/webrtc/issues/detail?id=6450) for me which also allows this on the sender side

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

Received on Thursday, 21 June 2018 06:04:24 UTC