Initial implementation of RtpTransportControllerReceive and related interfaces.

webrtc/call/flexfec_receive_stream_impl.h

 /*
  *  Copyright (c) 2016 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_CALL_FLEXFEC_RECEIVE_STREAM_IMPL_H_
 #define WEBRTC_CALL_FLEXFEC_RECEIVE_STREAM_IMPL_H_
 
 #include <memory>
 
 #include "webrtc/base/criticalsection.h"
 #include "webrtc/call/flexfec_receive_stream.h"
+#include "webrtc/call/rtp_transport_controller_receive.h"
+#include "webrtc/modules/rtp_rtcp/source/rtp_header_extension.h"
 
 namespace webrtc {
 
 class FlexfecReceiver;
 class ProcessThread;
 class ReceiveStatistics;
 class RecoveredPacketReceiver;
 class RtcpRttStats;
 class RtpPacketReceived;
 class RtpRtcp;
 
-class FlexfecReceiveStreamImpl : public FlexfecReceiveStream {
+class FlexfecReceiveStreamImpl : public FlexfecReceiveStream,
+                                 public RtpPacketReceiverInterface,
+                                 public RtpPacketSinkInterface {
  public:
   FlexfecReceiveStreamImpl(const Config& config,
                            RecoveredPacketReceiver* recovered_packet_receiver,
                            RtcpRttStats* rtt_stats,
                            ProcessThread* process_thread);
   ~FlexfecReceiveStreamImpl() override;
 
   const Config& GetConfig() const { return config_; }
 
-  // TODO(nisse): Intended to be part of an RtpPacketReceiver interface.
-  void OnRtpPacket(const RtpPacketReceived& packet);
+  // webrtc::RtpPacketReceiverInterface implementation, for the

[the sun, 2017/04/18 09:59:00]

What's the difference between an RtpPacketReceiver and an RtpPacketSink?

[nisse-webrtc, 2017/04/19 08:35:43]

A PacketReceiver knows the extensions mapping, and is responsible for calling
IdentifyExtensions. And hence needs a mutable packet object.

I'll investigate moving the responsibility for the extensions map up (or,
alternatively, eliminating IdentifyExtensions altogether). If that works out, we
only need the RtpPacketSink interface.

+  // FlexFEC stream.
+  bool OnRtpPacketReceive(RtpPacketReceived* packet) override;
+
+  // webrtc::RtpPacketSinkInterface implementation, for the media
+  // streams protected by FlexFEC.
+  void OnRtpPacket(const RtpPacketReceived& packet) override;
 
   // Implements FlexfecReceiveStream.
   void Start() override;
   void Stop() override;
   Stats GetStats() const override;
 
  private:
   // Config.
   const Config config_;
+  const RtpHeaderExtensionMap rtp_header_extensions_;
+
   bool started_ GUARDED_BY(crit_);
   rtc::CriticalSection crit_;
 
   // Erasure code interfacing.
   const std::unique_ptr<FlexfecReceiver> receiver_;
 
   // RTCP reporting.
   const std::unique_ptr<ReceiveStatistics> rtp_receive_statistics_;
   const std::unique_ptr<RtpRtcp> rtp_rtcp_;
   ProcessThread* process_thread_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_CALL_FLEXFEC_RECEIVE_STREAM_IMPL_H_