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

Unified Diff: webrtc/video/send_statistics_proxy_unittest.cc

Issue 2992043002: Renamed fields in common_types.h/RtcpStatistics. (Closed)
Patch Set: Rebase on new master 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
« no previous file with comments | « webrtc/video/report_block_stats_unittest.cc ('k') | webrtc/video/video_receive_stream.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video/send_statistics_proxy_unittest.cc
diff --git a/webrtc/video/send_statistics_proxy_unittest.cc b/webrtc/video/send_statistics_proxy_unittest.cc
index 4f09e48bd4408ab67134bbd9eb3a79e3c637cf45..99985c0f4dbd4885dd964d8de73b679fb85fcae9 100644
--- a/webrtc/video/send_statistics_proxy_unittest.cc
+++ b/webrtc/video/send_statistics_proxy_unittest.cc
@@ -133,9 +133,9 @@ class SendStatisticsProxyTest : public ::testing::Test {
EXPECT_EQ(a.rtp_stats.fec.packets, b.rtp_stats.fec.packets);
EXPECT_EQ(a.rtcp_stats.fraction_lost, b.rtcp_stats.fraction_lost);
- EXPECT_EQ(a.rtcp_stats.cumulative_lost, b.rtcp_stats.cumulative_lost);
- EXPECT_EQ(a.rtcp_stats.extended_max_sequence_number,
- b.rtcp_stats.extended_max_sequence_number);
+ EXPECT_EQ(a.rtcp_stats.packets_lost, b.rtcp_stats.packets_lost);
+ EXPECT_EQ(a.rtcp_stats.extended_highest_sequence_number,
+ b.rtcp_stats.extended_highest_sequence_number);
EXPECT_EQ(a.rtcp_stats.jitter, b.rtcp_stats.jitter);
}
}
@@ -157,8 +157,8 @@ TEST_F(SendStatisticsProxyTest, RtcpStatistics) {
// Add statistics with some arbitrary, but unique, numbers.
uint32_t offset = ssrc * sizeof(RtcpStatistics);
- ssrc_stats.rtcp_stats.cumulative_lost = offset;
- ssrc_stats.rtcp_stats.extended_max_sequence_number = offset + 1;
+ ssrc_stats.rtcp_stats.packets_lost = offset;
+ ssrc_stats.rtcp_stats.extended_highest_sequence_number = offset + 1;
ssrc_stats.rtcp_stats.fraction_lost = offset + 2;
ssrc_stats.rtcp_stats.jitter = offset + 3;
callback->StatisticsUpdated(ssrc_stats.rtcp_stats, ssrc);
@@ -168,8 +168,8 @@ TEST_F(SendStatisticsProxyTest, RtcpStatistics) {
// Add statistics with some arbitrary, but unique, numbers.
uint32_t offset = ssrc * sizeof(RtcpStatistics);
- ssrc_stats.rtcp_stats.cumulative_lost = offset;
- ssrc_stats.rtcp_stats.extended_max_sequence_number = offset + 1;
+ ssrc_stats.rtcp_stats.packets_lost = offset;
+ ssrc_stats.rtcp_stats.extended_highest_sequence_number = offset + 1;
ssrc_stats.rtcp_stats.fraction_lost = offset + 2;
ssrc_stats.rtcp_stats.jitter = offset + 3;
callback->StatisticsUpdated(ssrc_stats.rtcp_stats, ssrc);
« no previous file with comments | « webrtc/video/report_block_stats_unittest.cc ('k') | webrtc/video/video_receive_stream.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698