Re: [webrtc-pc] Add pc.restartIce() method. (#2169)

I have a working webrtc.org implementation of what I think this PR intends to do, [code here](https://webrtc-review.googlesource.com/c/src/+/144941). @jan-ivar is this correct?
1. In both SLD(answer) and SRD(answer) I am comparing the set description with the current ICE credentials, based on looking at [[CurrentLocalDescription]].
2. Because restartIce() does not take signalingState into account, this causes the [[RestartIce]] slot to reset when the O/A finishes, even if restartIce() was called after createOffer(). I am wondering [if sometimes we need to restart ICE twice](https://webrtc-review.googlesource.com/c/src/+/144941/1/pc/peer_connection_ice_unittest.cc#887)?

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

Received on Tuesday, 9 July 2019 13:18:10 UTC