Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(485)

Unified Diff: webrtc/modules/rtp_rtcp/source/flexfec_sender.cc

Issue 2441613002: Add FlexfecSender. (Closed)
Patch Set: Rebase. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/modules/rtp_rtcp/rtp_rtcp.gypi ('k') | webrtc/modules/rtp_rtcp/source/flexfec_sender_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
diff --git a/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc b/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
new file mode 100644
index 0000000000000000000000000000000000000000..9f04161e524b3482e1dd22580ff780b168d73d53
--- /dev/null
+++ b/webrtc/modules/rtp_rtcp/source/flexfec_sender.cc
@@ -0,0 +1,165 @@
+/*
+ * 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.
+ */
+
+#include "webrtc/modules/rtp_rtcp/include/flexfec_sender.h"
+
+#include <utility>
+
+#include "webrtc/base/logging.h"
+#include "webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h"
+#include "webrtc/modules/rtp_rtcp/source/forward_error_correction.h"
+#include "webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h"
+
+namespace webrtc {
+
+namespace {
+
+// Let first sequence number be in the first half of the interval.
+constexpr uint16_t kMaxInitRtpSeqNumber = 0x7fff;
+
+// See breakdown in flexfec_header_reader_writer.cc.
+constexpr size_t kFlexfecMaxHeaderSize = 32;
+
+// Since we will mainly use FlexFEC to protect video streams, we use a 90 kHz
+// clock for the RTP timestamps. (This is according to the RFC, which states
+// that it is RECOMMENDED to use the same clock frequency for FlexFEC as for
+// the protected media stream.)
+// The constant converts from clock millisecond timestamps to the 90 kHz
+// RTP timestamp.
+const int kMsToRtpTimestamp = kVideoPayloadTypeFrequency / 1000;
+
+// How often to log the generated FEC packets to the text log.
+constexpr int64_t kPacketLogIntervalMs = 10000;
+
+} // namespace
+
+FlexfecSender::FlexfecSender(
+ int payload_type,
+ uint32_t ssrc,
+ uint32_t protected_media_ssrc,
+ const std::vector<RtpExtension>& rtp_header_extensions,
+ Clock* clock)
+ : clock_(clock),
+ random_(clock_->TimeInMicroseconds()),
+ last_generated_packet_ms_(-1),
+ payload_type_(payload_type),
+ // Initialize the timestamp offset and RTP sequence numbers randomly.
+ // (This is not intended to be cryptographically strong.)
+ timestamp_offset_(random_.Rand<uint32_t>()),
+ ssrc_(ssrc),
+ protected_media_ssrc_(protected_media_ssrc),
+ seq_num_(random_.Rand(1, kMaxInitRtpSeqNumber)),
+ ulpfec_generator_(ForwardErrorCorrection::CreateFlexfec()),
+ rtp_header_extension_map_() {
+ // This object should not have been instantiated if FlexFEC is disabled.
+ RTC_DCHECK_GE(payload_type, 0);
+ RTC_DCHECK_LE(payload_type, 127);
+
+ // It's OK to create this object on a different thread/task queue than
+ // the one used during main operation.
+ sequence_checker_.Detach();
+
+ // Register RTP header extensions for BWE.
+ for (const auto& extension : rtp_header_extensions) {
+ if (extension.uri == RtpExtension::kTransportSequenceNumberUri) {
+ rtp_header_extension_map_.Register(kRtpExtensionTransportSequenceNumber,
+ extension.id);
+ } else if (extension.uri == RtpExtension::kAbsSendTimeUri) {
+ rtp_header_extension_map_.Register(kRtpExtensionAbsoluteSendTime,
+ extension.id);
+ } else if (extension.uri == RtpExtension::kTimestampOffsetUri) {
+ rtp_header_extension_map_.Register(kRtpExtensionTransmissionTimeOffset,
+ extension.id);
+ } else {
+ LOG(LS_WARNING) << "RTP header extension with id: " << extension.id
+ << ", uri: " << extension.uri
+ << ", is unsupported by FlexfecSender.";
+ }
+ }
+}
+
+FlexfecSender::~FlexfecSender() = default;
+
+// We are reusing the implementation from UlpfecGenerator for SetFecParameters,
+// AddRtpPacketAndGenerateFec, and FecAvailable.
+void FlexfecSender::SetFecParameters(const FecProtectionParams& params) {
+ RTC_DCHECK_CALLED_SEQUENTIALLY(&sequence_checker_);
+ ulpfec_generator_.SetFecParameters(&params);
+}
+
+bool FlexfecSender::AddRtpPacketAndGenerateFec(
+ const RtpPacketToSend& packet) {
+ RTC_DCHECK_CALLED_SEQUENTIALLY(&sequence_checker_);
+ // TODO(brandtr): Generalize this SSRC check when we support multistream
+ // protection.
+ RTC_DCHECK_EQ(packet.Ssrc(), protected_media_ssrc_);
+ return ulpfec_generator_.AddRtpPacketAndGenerateFec(
+ packet.data(), packet.payload_size(), packet.headers_size()) == 0;
+}
+
+bool FlexfecSender::FecAvailable() const {
+ RTC_DCHECK_CALLED_SEQUENTIALLY(&sequence_checker_);
+ return ulpfec_generator_.FecAvailable();
+}
+
+std::vector<std::unique_ptr<RtpPacketToSend>>
+FlexfecSender::GetFecPackets() {
+ RTC_DCHECK_CALLED_SEQUENTIALLY(&sequence_checker_);
+
+ std::vector<std::unique_ptr<RtpPacketToSend>> fec_packets_to_send;
+ fec_packets_to_send.reserve(ulpfec_generator_.generated_fec_packets_.size());
+ for (const auto& fec_packet : ulpfec_generator_.generated_fec_packets_) {
+ std::unique_ptr<RtpPacketToSend> fec_packet_to_send(
+ new RtpPacketToSend(&rtp_header_extension_map_));
+
+ // RTP header.
+ fec_packet_to_send->SetMarker(false);
+ fec_packet_to_send->SetPayloadType(payload_type_);
+ fec_packet_to_send->SetSequenceNumber(seq_num_++);
+ fec_packet_to_send->SetTimestamp(
+ timestamp_offset_ +
+ static_cast<uint32_t>(kMsToRtpTimestamp *
+ clock_->TimeInMilliseconds()));
+ // Set "capture time" so that the TransmissionOffset header extension
+ // can be set by the RTPSender.
+ fec_packet_to_send->set_capture_time_ms(clock_->TimeInMilliseconds());
+ fec_packet_to_send->SetSsrc(ssrc_);
+ // Reserve extensions, if registered. These will be set by the RTPSender.
+ fec_packet_to_send->ReserveExtension<AbsoluteSendTime>();
+ fec_packet_to_send->ReserveExtension<TransmissionOffset>();
+ fec_packet_to_send->ReserveExtension<TransportSequenceNumber>();
+
+ // RTP payload.
+ uint8_t* payload = fec_packet_to_send->AllocatePayload(fec_packet->length);
+ memcpy(payload, fec_packet->data, fec_packet->length);
+
+ fec_packets_to_send.push_back(std::move(fec_packet_to_send));
+ }
+ ulpfec_generator_.ResetState();
+
+ // TODO(brandtr): Remove this log output when the FlexFEC subsystem is
+ // properly wired up in a robust way.
+ int64_t now_ms = clock_->TimeInMilliseconds();
+ if (!fec_packets_to_send.empty() &&
+ now_ms - last_generated_packet_ms_ > kPacketLogIntervalMs) {
+ LOG(LS_INFO) << "Generated " << fec_packets_to_send.size()
+ << " FlexFEC packets with payload type: " << payload_type_
+ << " and SSRC: " << ssrc_ << ".";
+ last_generated_packet_ms_ = now_ms;
+ }
+
+ return fec_packets_to_send;
+}
+
+size_t FlexfecSender::MaxPacketOverhead() const {
+ return kFlexfecMaxHeaderSize;
+}
+
+} // namespace webrtc
« no previous file with comments | « webrtc/modules/rtp_rtcp/rtp_rtcp.gypi ('k') | webrtc/modules/rtp_rtcp/source/flexfec_sender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698