A call to RTCRtpSender.replaceTrack() [1] eventually ends up in MediaEndpointPeerConnection::replaceTrack(). [1] https://w3c.github.io/webrtc-pc/archives/20160513/webrtc.html#dom-rtcrtpsender-replacetrack
Created attachment 281166 [details] Proposed patch
Comment on attachment 281166 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=281166&action=review > Source/WebCore/Modules/mediastream/MediaEndpointPeerConnection.cpp:634 > + const String& mid = transceiver->mid(); Is it worth ASSERTing transceiver here?
(In reply to comment #2) > Comment on attachment 281166 [details] > Proposed patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=281166&action=review > > > Source/WebCore/Modules/mediastream/MediaEndpointPeerConnection.cpp:634 > > + const String& mid = transceiver->mid(); > > Is it worth ASSERTing transceiver here? We can do that to emphasize that it should always be found or something is broken.
Created attachment 281203 [details] Patch for landing
Comment on attachment 281203 [details] Patch for landing Thanks for reviewing Eric!
Comment on attachment 281203 [details] Patch for landing Clearing flags on attachment: 281203 Committed r202026: <http://trac.webkit.org/changeset/202026>