Protected streams report RTP messages directly to the FlexFec streams

webrtc/modules/rtp_rtcp/source/flexfec_receiver.cc

Index: webrtc/modules/rtp_rtcp/source/flexfec_receiver.cc
diff --git a/webrtc/modules/rtp_rtcp/source/flexfec_receiver.cc b/webrtc/modules/rtp_rtcp/source/flexfec_receiver.cc
index 13899d5e5feec54a1f9d2b95a606a452d736d184..870fe4ce1747b3118048eaef2d99819d72f393dc 100644
--- a/webrtc/modules/rtp_rtcp/source/flexfec_receiver.cc
+++ b/webrtc/modules/rtp_rtcp/source/flexfec_receiver.cc
@@ -59,6 +59,9 @@ FecPacketCounter FlexfecReceiver::GetPacketCounter() const {
   return packet_counter_;
 }
 
+// TODO(eladalon): We should be able to avoid FlexFEC-recovered packets
+// reaching back here, by making the OnRtpPacket() function take a parameter
+// such as |is_recovered|, which would be |true| for FlexFEC-recovered packets.

[stefan-webrtc, 2017/07/07 09:36:24]

Seems a bit ugly to me since all rtp packet sinks would have to have this flag,
but only flex fec would care about it?

[eladalon, 2017/07/07 14:16:24]

I see now that RtpPacketReceived::recovered() exists. I've changed to TODO
accordingly.

 bool FlexfecReceiver::AddReceivedPacket(const RtpPacketReceived& packet) {
   RTC_DCHECK_CALLED_SEQUENTIALLY(&sequence_checker_);