(Attempt to) Implement ICE Restarts

[soup-in-boots]

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update [ ] Bug fix [X] New feature [ ] Other, please explain:

What changes did you make? (Give an overview) Added an iceRestartEnabled option; when set to a truthy value, will prevent transitions from to the "failed" state from destroying the peer. Further, if enabled and the failed state is reached, will automatically initiate an ICE restart.

Added a restart function which, when called by the initiator, will invoke restartIce on the underlying RTCPeerConnection.

Which issue (if any) does this pull request address? #579

Is there anything you'd like reviewers to focus on? Advice for test cases. Additionally, I don't know the broader implications of these changes, so I'll be relying on you all to guide me in that respect.

[soup-in-boots]

For the record, the build is failing in 14.x due to a bug in wrtc: https://github.com/node-webrtc/node-webrtc/issues/636

[DJFliX]

Hi! I'm interested in using this functionality in simple-peer. Would it help if I use the changes @fauxsoup made and create an MR that is mergable?

[soup-in-boots]

I have resolved the merge conflict. Can I get some feedback on this? I would love to get this up to your standards and merged!

[lofcz]

cc @feross

[sooryranga]

cc @feross

[soup-in-boots]

Rebased onto the latest version. Please let me know if there's anything more I can do.

[michalzaq12]

@fauxsoup I'm waiting for this feature. If there's anything you need help with to make this PR successful, let me know!