Ignore multiple MSID's as preparation for Transceivers

RESOLVED FIXED in Firefox 56

Status

()

Core
WebRTC: Signaling
P1
normal
RESOLVED FIXED
a year ago
11 months ago

People

(Reporter: drno, Unassigned)

Tracking

Trunk
mozilla56
Points:
---

Firefox Tracking Flags

(firefox56 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

a year ago
Byron pointed out that since the new Transceivers are going to allow multiple MSID's per m-section current releases of Firefox might reject SDP's from future Transceiver enabled Firefoxes.

E.g. http://searchfox.org/mozilla-central/rev/c49a70b53f67dd5550eec8a08793805f2aca8d42/media/webrtc/signaling/src/sdp/SdpHelper.cpp#493

Thus we should make this behavior less strict by simply using the first MSID and ignore all other MSID's. That should hopefully help to make older Firefoxes interop with future versions with Transceivers.
Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8877879 [details]
Bug 1373144: turn multiple msid's error into warning.

https://reviewboard.mozilla.org/r/149306/#review153986
Attachment #8877879 - Flags: review?(docfaraday) → review+

Comment 3

a year ago
Pushed by drno@ohlmeier.org:
https://hg.mozilla.org/integration/autoland/rev/29d5b5f6514c
turn multiple msid's error into warning. r=bwc

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/29d5b5f6514c
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.