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

Unified Diff: webrtc/modules/rtp_rtcp/test/testAPI/test_api_rtcp.cc

Issue 2994633002: Renamed fields in rtp_rtcp_defines.h/RTCPReportBlock (Closed)
Patch Set: Created 3 years, 4 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
Index: webrtc/modules/rtp_rtcp/test/testAPI/test_api_rtcp.cc
diff --git a/webrtc/modules/rtp_rtcp/test/testAPI/test_api_rtcp.cc b/webrtc/modules/rtp_rtcp/test/testAPI/test_api_rtcp.cc
index a124f6272548f90f11e54cdcf7d5276956c156c0..3ce1af78599a93888a46f20c1fd8f325eb37527b 100644
--- a/webrtc/modules/rtp_rtcp/test/testAPI/test_api_rtcp.cc
+++ b/webrtc/modules/rtp_rtcp/test/testAPI/test_api_rtcp.cc
@@ -235,13 +235,14 @@ TEST_F(RtpRtcpRtcpTest, RemoteRTCPStatRemote) {
ASSERT_EQ(1u, report_blocks.size());
// |test_ssrc+1| is the SSRC of module2 that send the report.
- EXPECT_EQ(test_ssrc+1, report_blocks[0].remoteSSRC);
- EXPECT_EQ(test_ssrc, report_blocks[0].sourceSSRC);
-
- EXPECT_EQ(0u, report_blocks[0].cumulativeLost);
- EXPECT_LT(0u, report_blocks[0].delaySinceLastSR);
- EXPECT_EQ(test_sequence_number, report_blocks[0].extendedHighSeqNum);
- EXPECT_EQ(0u, report_blocks[0].fractionLost);
+ EXPECT_EQ(test_ssrc + 1, report_blocks[0].sender_ssrc);
+ EXPECT_EQ(test_ssrc, report_blocks[0].source_ssrc);
+
+ EXPECT_EQ(0u, report_blocks[0].packets_lost);
+ EXPECT_LT(0u, report_blocks[0].delay_since_last_sender_report);
+ EXPECT_EQ(test_sequence_number,
+ report_blocks[0].extended_highest_sequence_number);
+ EXPECT_EQ(0u, report_blocks[0].fraction_lost);
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698