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

Unified Diff: webrtc/call/rtx_receive_stream.cc

Issue 3012963002: Reland of Use RtxReceiveStream. (Closed)
Patch Set: Fix receive stream configuration in video quality test. Created 3 years, 3 months 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/call/rtx_receive_stream.h ('k') | webrtc/call/video_receive_stream.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/call/rtx_receive_stream.cc
diff --git a/webrtc/call/rtx_receive_stream.cc b/webrtc/call/rtx_receive_stream.cc
index 16463525c729de224d6a1dfb9befe76d4617a25f..6a5432fea6576965d50545b25974b7e3d27a6fb6 100644
--- a/webrtc/call/rtx_receive_stream.cc
+++ b/webrtc/call/rtx_receive_stream.cc
@@ -11,17 +11,21 @@
#include <utility>
#include "webrtc/call/rtx_receive_stream.h"
+#include "webrtc/modules/rtp_rtcp/include/receive_statistics.h"
#include "webrtc/modules/rtp_rtcp/source/rtp_packet_received.h"
#include "webrtc/rtc_base/logging.h"
namespace webrtc {
-RtxReceiveStream::RtxReceiveStream(RtpPacketSinkInterface* media_sink,
- std::map<int, int> associated_payload_types,
- uint32_t media_ssrc)
+RtxReceiveStream::RtxReceiveStream(
+ RtpPacketSinkInterface* media_sink,
+ std::map<int, int> associated_payload_types,
+ uint32_t media_ssrc,
+ ReceiveStatistics* rtp_receive_statistics /* = nullptr */)
: media_sink_(media_sink),
associated_payload_types_(std::move(associated_payload_types)),
- media_ssrc_(media_ssrc) {
+ media_ssrc_(media_ssrc),
+ rtp_receive_statistics_(rtp_receive_statistics) {
if (associated_payload_types_.empty()) {
LOG(LS_WARNING)
<< "RtxReceiveStream created with empty payload type mapping.";
@@ -31,6 +35,12 @@ RtxReceiveStream::RtxReceiveStream(RtpPacketSinkInterface* media_sink,
RtxReceiveStream::~RtxReceiveStream() = default;
void RtxReceiveStream::OnRtpPacket(const RtpPacketReceived& rtx_packet) {
+ if (rtp_receive_statistics_) {
+ RTPHeader header;
+ rtx_packet.GetHeader(&header);
+ rtp_receive_statistics_->IncomingPacket(header, rtx_packet.size(),
+ false /* retransmitted */);
+ }
rtc::ArrayView<const uint8_t> payload = rtx_packet.payload();
if (payload.size() < kRtxHeaderSize) {
« no previous file with comments | « webrtc/call/rtx_receive_stream.h ('k') | webrtc/call/video_receive_stream.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698