Closed Bug 1486012 Opened 6 years ago Closed 6 years ago

re-implement ICE restart as the creation of new ICE streams on the pre-existing context

Categories

(Core :: WebRTC: Networking, enhancement, P2)

60 Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: bwc, Assigned: bwc)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

In order to make it possible to perform partial ICE restarts, we need to change the way we handle ICE restart. Right now, when an ICE restart happens, we create a whole new ICE context, and then switch over to that new context once ICE succeeds on it. What we should instead be doing is creating extra ICE streams on the context we already have, and then removing the old ones once we're ready.

We probably want to do this before we try to move mtransport into a separate process, because this is a simplification in the mtransport API.

This would mean keying ICE streams based on a combination of transport-id and ufrag/pwd, at least until we use unique ufrag/pwds for each ICE stream.
Status: NEW → ASSIGNED
Comment on attachment 9009226 [details]
Bug 1486012: Rework ICE restart.

Michael Froman [:mjf] has approved the revision.
Attachment #9009226 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/f2da055061b9
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Depends on: 1493146
This change is breaking interop with Safari and Chrome! 
It looks to me like Firefox is generating new ICE username and password in it's answer, which the other browsers don't seem to like at all (haven't checked the specs yet).
Flags: needinfo?(docfaraday)
Never mind. It looks like bug 1493146 will fix the problem I'm seeing with Nightly right now. But bug 1493146 hasn't made it into my current Nightly.
Flags: needinfo?(docfaraday)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: