1373144 - Ignore multiple MSID's as preparation for Transceivers

Status: RESOLVED FIXED

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

Description

[Nils Ohlmeier [:drno]]

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 1

[Nils Ohlmeier [:drno]]

Attachment: Bug 1373144: turn multiple msid's error into warning.

Review commit: https://reviewboard.mozilla.org/r/149306/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/149306/

Comment 2

[Byron Campen [:bwc]]

Comment on attachment 8877879 [details]
Bug 1373144: turn multiple msid's error into warning.

https://reviewboard.mozilla.org/r/149306/#review153986

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/29d5b5f6514c
turn multiple msid's error into warning. r=bwc

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/29d5b5f6514c