Closed Bug 1392745 Opened 7 years ago Closed 7 years ago

Keep MID in disabled m-sections

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: drno, Assigned: drno)

References

Details

Attachments

(2 files)

It makes sense to maintain the MID's in m-section when we disable them, so that the offerer can still use the MID to figure out which m-section we disabled.
Attachment #8899984 - Flags: review?(mfroman)
Attachment #8899985 - Flags: review?(mfroman)
Comment on attachment 8899984 [details]
Bug 1392745: keep MID when disabling m-sections

https://reviewboard.mozilla.org/r/171298/#review176944

Looks good to me.
Attachment #8899984 - Flags: review?(mfroman) → review+
Comment on attachment 8899985 [details]
Bug 1392745: verify mid presence in inactive m-section

https://reviewboard.mozilla.org/r/171300/#review176950

Looks good to me.
Attachment #8899985 - Flags: review?(mfroman) → review+
Pushed by drno@ohlmeier.org:
https://hg.mozilla.org/integration/autoland/rev/5ce80881b78c
keep MID when disabling m-sections r=mjf
https://hg.mozilla.org/integration/autoland/rev/58633245c138
verify mid presence in inactive m-section r=mjf
Blocks: 1396974
No longer blocks: 1396974
Depends on: 1396974
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: