Created attachment 353860 [details] Script that reproduces the issue. Just run it in Safari 12.1 console with "Unified-Plan" enabled. Scenario: * Have a pc with an audio transceiver for just sending. * Do transceiver.sender.replaceTrack(null) and renegotiate. * Do transceiver.sender.replaceTrack(track) and renegotiate. After that, we have two audio transceivers so two m=audio sections. Of course this should not happen. It works fine in Chrome and Firefox. Script attached.
This is a left over of plan B/unified plan migration. Will fix it
Created attachment 353952 [details] Patch
Comment on attachment 353952 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=353952&action=review > Source/WebCore/ChangeLog:11 > + Covered by updated test. Nit: you should mention that you updated webrtc/video-addTransceiver.html to test this.
Created attachment 354040 [details] Patch for landing
Comment on attachment 354040 [details] Patch for landing Clearing flags on attachment: 354040 Committed r237910: <https://trac.webkit.org/changeset/237910>
All reviewed patches have been landed. Closing bug.
<rdar://problem/45864192>
(In reply to Eric Carlson from comment #3) > Comment on attachment 353952 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=353952&action=review > > > Source/WebCore/ChangeLog:11 > > + Covered by updated test. > > Nit: you should mention that you updated webrtc/video-addTransceiver.html to > test this. Thanks for the review. Done!