Re: [webrtc-pc] Block replaceTrack on pc.close() like the others, not tc.stop(). (#2107)

> Only question is whether the `replaceTrack` promise should resolve or be rejected.

I think the PR is correct; the promise should be resolved because the track was in-fact replaced. Transmission of that track is a different operation, and even a successful replaceTrack does not guarantee that the new frames reaches the other endpoint. You could be pedantic and update "have the sender switch seamlessly to transmitting withTrack instead of the sender's existing track" to clarify that transmission only occurs if the link is still open, but I don't think it's necessary.

-- 
GitHub Notification of comment by henbos
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/pull/2107#issuecomment-466014575 using your GitHub account

Received on Thursday, 21 February 2019 14:15:22 UTC