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

Unified Diff: webrtc/test/rtcp_packet_parser.cc

Issue 2372113005: Replace RTCPUtility RTCPParser with TestRtcpParser (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/test/rtcp_packet_parser.h ('k') | webrtc/video/end_to_end_tests.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/test/rtcp_packet_parser.cc
diff --git a/webrtc/test/rtcp_packet_parser.cc b/webrtc/test/rtcp_packet_parser.cc
index 2eb48c11a37b99ef44a9b6599504b05032194c18..b9cf6db1a737f1d63341a7c2e36c4f17999f365f 100644
--- a/webrtc/test/rtcp_packet_parser.cc
+++ b/webrtc/test/rtcp_packet_parser.cc
@@ -36,10 +36,10 @@ bool RtcpPacketParser::Parse(const void* data, size_t length) {
app_.Parse(header);
break;
case rtcp::Bye::kPacketType:
- bye_.Parse(header);
+ bye_.Parse(header, &sender_ssrc_);
break;
case rtcp::ExtendedReports::kPacketType:
- xr_.Parse(header);
+ xr_.Parse(header, &sender_ssrc_);
break;
case rtcp::ExtendedJitterReport::kPacketType:
ij_.Parse(header);
@@ -47,19 +47,19 @@ bool RtcpPacketParser::Parse(const void* data, size_t length) {
case rtcp::Psfb::kPacketType:
switch (header.fmt()) {
case rtcp::Fir::kFeedbackMessageType:
- fir_.Parse(header);
+ fir_.Parse(header, &sender_ssrc_);
break;
case rtcp::Pli::kFeedbackMessageType:
- pli_.Parse(header);
+ pli_.Parse(header, &sender_ssrc_);
break;
case rtcp::Remb::kFeedbackMessageType:
- remb_.Parse(header);
+ remb_.Parse(header, &sender_ssrc_);
break;
case rtcp::Rpsi::kFeedbackMessageType:
- rpsi_.Parse(header);
+ rpsi_.Parse(header, &sender_ssrc_);
break;
case rtcp::Sli::kFeedbackMessageType:
- sli_.Parse(header);
+ sli_.Parse(header, &sender_ssrc_);
break;
default:
LOG(LS_WARNING) << "Unknown rtcp payload specific feedback type "
@@ -68,24 +68,24 @@ bool RtcpPacketParser::Parse(const void* data, size_t length) {
}
break;
case rtcp::ReceiverReport::kPacketType:
- receiver_report_.Parse(header);
+ receiver_report_.Parse(header, &sender_ssrc_);
break;
case rtcp::Rtpfb::kPacketType:
switch (header.fmt()) {
case rtcp::Nack::kFeedbackMessageType:
- nack_.Parse(header);
+ nack_.Parse(header, &sender_ssrc_);
break;
case rtcp::RapidResyncRequest::kFeedbackMessageType:
- rrr_.Parse(header);
+ rrr_.Parse(header, &sender_ssrc_);
break;
case rtcp::Tmmbn::kFeedbackMessageType:
- tmmbn_.Parse(header);
+ tmmbn_.Parse(header, &sender_ssrc_);
break;
case rtcp::Tmmbr::kFeedbackMessageType:
- tmmbr_.Parse(header);
+ tmmbr_.Parse(header, &sender_ssrc_);
break;
case rtcp::TransportFeedback::kFeedbackMessageType:
- transport_feedback_.Parse(header);
+ transport_feedback_.Parse(header, &sender_ssrc_);
break;
default:
LOG(LS_WARNING) << "Unknown rtcp transport feedback type "
@@ -97,7 +97,7 @@ bool RtcpPacketParser::Parse(const void* data, size_t length) {
sdes_.Parse(header);
break;
case rtcp::SenderReport::kPacketType:
- sender_report_.Parse(header);
+ sender_report_.Parse(header, &sender_ssrc_);
break;
default:
LOG(LS_WARNING) << "Unknown rtcp packet type " << header.type();
« no previous file with comments | « webrtc/test/rtcp_packet_parser.h ('k') | webrtc/video/end_to_end_tests.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698