[Webkit-unassigned] [Bug 198402] iOS 12.3 and 12.4 emit PeerConnection SDP offer m-lines in random order

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri May 31 11:06:40 PDT 2019


youenn fablet <youennf at gmail.com> changed:

           What    |Removed                     |Added
             Status|NEW                         |RESOLVED
         Resolution|---                         |CONFIGURATION CHANGED

--- Comment #4 from youenn fablet <youennf at gmail.com> ---
(In reply to Chad Phillips from comment #3)
> youenn,
> Thanks for your replies, very helpful!
> Two things:
> 1. Adding the media tracks via addTrack() in the specific order I want does
> indeed fix the issue.


> 2. The munged SDP was failing because it was missing a trailing \r\n after
> the last line. This seems like a silly reason to fail SDP parsing, perhaps
> that can be rectified? At the very least, an SDP parsing error should
> include some more helpful output for the developer!

You are not the first one caught up in this area.
This error message is usually a signal of such kind of error, agreed it is not very clear though.

> 3. The adapter.js 'addStream' shim for Safari uses stream.getTracks()
> internally to then call addTrack() on all the tracks in the stream. Is it
> possible that the Webkit implementation of getTracks() returns the media
> tracks in random order?

Tracks are stored in a map internally whose key is the stream id.
But this is implementation dependent and the spec does not require any specific order, nor should implementations rely on that.
adapter.js could be of some help there by doing getAudioTracks/getVideoTracks so as to preserve the old behavior.

I'll close the bug, please reopen it if needed.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20190531/55c4a7a0/attachment.html>

More information about the webkit-unassigned mailing list