Chromium Code Reviews
DescriptionMerge to 60: Avoid toggling default receive streams in WebRtcVideoChannel2.
This CL removes |default_recv_ssrc_| from DefaultUnsignalledSsrcHandler
and replaces it with calls to a new member function
WebRtcVideoChannel2::GetDefaultReceiveStreamSsrc. The latter checks
the |default_stream_| member on the
WebRtcVideoChannel2::WebRtcVideoReceiveStreams to know which stream
is the current default stream.
This change removes duplicate state and fixes an issue where
incoming unsignaled SSRCs would compete for being the default
receive stream.
BUG=webrtc:7725, chromium:731679
TBR=sprang@webrtc.org, deadbeef@webrtc.org
NOTRY=True
NOPRESUBMIT=True
Review-Url (original): https://codereview.webrtc.org/2906893002
Cr-Commit-Position (original): refs/heads/master@{#18314}
Review-Url: https://codereview.webrtc.org/2943343002
Cr-Commit-Position: refs/branch-heads/60@{#3}
Cr-Branched-From: c61bf947b4ac31f3500858ffcae6fee39d799930-refs/heads/master@{#18252}
Committed: https://chromium.googlesource.com/external/webrtc/+/9692b879044aadbee4430d2ebf69c3e61d34f40a
Patch Set 1 #
Messages
Total messages: 6 (4 generated)
|
|||||||||||||||||||||||||||||||||||||