Protected streams report RTP messages directly to the FlexFec streams

webrtc/video_receive_stream.h

 /*
  *  Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #ifndef WEBRTC_VIDEO_RECEIVE_STREAM_H_
 #define WEBRTC_VIDEO_RECEIVE_STREAM_H_
 
 #include <limits>
 #include <map>
 #include <string>
 #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"
 #include "webrtc/media/base/videosinkinterface.h"
 #include "webrtc/rtc_base/platform_file.h"
 
 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.
   struct Decoder {
     std::string ToString() const;
 
     // The actual decoder instance.
     VideoDecoder* decoder = nullptr;
 
     // Received RTP packets with this payload type will be sent to this decoder
@@ skipping 180 matching lines @@
     EnableEncodedFrameRecording(rtc::kInvalidPlatformFileValue, 0);
   }
 
  protected:
   virtual ~VideoReceiveStream() {}
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_VIDEO_RECEIVE_STREAM_H_