Protected streams report RTP messages directly to the FlexFec streams

webrtc/video_receive_stream.h

Index: webrtc/video_receive_stream.h
diff --git a/webrtc/video_receive_stream.h b/webrtc/video_receive_stream.h
index 863cb2e088267b8dfabfb1fd98b187501c1faf02..ba2b32db0c1ab61fcdf88f185f29c66205497f71 100644
--- a/webrtc/video_receive_stream.h
+++ b/webrtc/video_receive_stream.h
@@ -17,6 +17,9 @@
 #include <vector>
 
 #include "webrtc/api/call/transport.h"
+// TODO(eladalon): !!! Good? Bad? Do we want to have these functions explicitly
+// part of the interface, without inheriting them, perhaps?

[danilchap, 2017/07/24 09:03:46]

I do not see a value in an interface that no code use.

[eladalon, 2017/07/24 13:15:48]

Acknowledged.

+#include "webrtc/call/secondary_rtp_sinks_container.h"
 #include "webrtc/common_types.h"
 #include "webrtc/common_video/include/frame_callback.h"
 #include "webrtc/config.h"
@@ -27,7 +30,7 @@ namespace webrtc {
 
 class VideoDecoder;
 
-class VideoReceiveStream {
+class VideoReceiveStream : public SecondaryRtpSinksContainer {
  public:
   // TODO(mflodman) Move all these settings to VideoDecoder and move the
   // declaration to common_types.h.