SSRC and RSID may only refer to one sink each in RtpDemuxer

webrtc/call/rtp_stream_receiver_controller.cc

Index: webrtc/call/rtp_stream_receiver_controller.cc
diff --git a/webrtc/call/rtp_stream_receiver_controller.cc b/webrtc/call/rtp_stream_receiver_controller.cc
index a4b1e36ae262c1951479c36b57e07ea71a76041d..743ae02254831c788967be363956b798ce34ae5b 100644
--- a/webrtc/call/rtp_stream_receiver_controller.cc
+++ b/webrtc/call/rtp_stream_receiver_controller.cc
@@ -46,7 +46,7 @@ bool RtpStreamReceiverController::OnRtpPacket(const RtpPacketReceived& packet) {
 void RtpStreamReceiverController::AddSink(uint32_t ssrc,
                                           RtpPacketSinkInterface* sink) {
   rtc::CritScope cs(&lock_);
-  return demuxer_.AddSink(ssrc, sink);
+  demuxer_.AddSink(ssrc, sink);  // TODO(eladalon): Return-value useful here?

[eladalon, 2017/06/30 12:53:24]

Reviewers - input welcome!

[danilchap, 2017/06/30 16:45:31]

might be helpful.
either return bool, or RTC_CHECK() sink was added.

Otherwise if AddSink will fail it will be hard to understand what's going on.

[eladalon, 2017/07/03 08:23:35]

Done.

 }
 
 size_t RtpStreamReceiverController::RemoveSink(