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

Unified Diff: webrtc/video/report_block_stats_unittest.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/video/report_block_stats_unittest.cc
diff --git a/webrtc/video/report_block_stats_unittest.cc b/webrtc/video/report_block_stats_unittest.cc
index d8bd7be49b2e28de901e14001dcd0c2c06c838db..3ce49ee6dbed157c4a20057eeedfb63a9f9cee9e 100644
--- a/webrtc/video/report_block_stats_unittest.cc
+++ b/webrtc/video/report_block_stats_unittest.cc
@@ -19,32 +19,32 @@ class ReportBlockStatsTest : public ::testing::Test {
void SetUp() override {
// kSsrc1: block 1-3.
- block1_1_.cumulativeLost = 10;
- block1_1_.fractionLost = 123;
- block1_1_.extendedHighSeqNum = 24000;
+ block1_1_.packets_lost = 10;
+ block1_1_.fraction_lost = 123;
+ block1_1_.extended_highest_sequence_number = 24000;
block1_1_.jitter = 777;
- block1_1_.sourceSSRC = kSsrc1;
- block1_2_.cumulativeLost = 15;
- block1_2_.fractionLost = 0;
- block1_2_.extendedHighSeqNum = 24100;
+ block1_1_.source_ssrc = kSsrc1;
+ block1_2_.packets_lost = 15;
+ block1_2_.fraction_lost = 0;
+ block1_2_.extended_highest_sequence_number = 24100;
block1_2_.jitter = 222;
- block1_2_.sourceSSRC = kSsrc1;
- block1_3_.cumulativeLost = 50;
- block1_3_.fractionLost = 0;
- block1_3_.extendedHighSeqNum = 24200;
+ block1_2_.source_ssrc = kSsrc1;
+ block1_3_.packets_lost = 50;
+ block1_3_.fraction_lost = 0;
+ block1_3_.extended_highest_sequence_number = 24200;
block1_3_.jitter = 333;
- block1_3_.sourceSSRC = kSsrc1;
+ block1_3_.source_ssrc = kSsrc1;
// kSsrc2: block 1,2.
- block2_1_.cumulativeLost = 111;
- block2_1_.fractionLost = 222;
- block2_1_.extendedHighSeqNum = 8500;
+ block2_1_.packets_lost = 111;
+ block2_1_.fraction_lost = 222;
+ block2_1_.extended_highest_sequence_number = 8500;
block2_1_.jitter = 555;
- block2_1_.sourceSSRC = kSsrc2;
- block2_2_.cumulativeLost = 136;
- block2_2_.fractionLost = 0;
- block2_2_.extendedHighSeqNum = 8800;
+ block2_1_.source_ssrc = kSsrc2;
+ block2_2_.packets_lost = 136;
+ block2_2_.fraction_lost = 0;
+ block2_2_.extended_highest_sequence_number = 8800;
block2_2_.jitter = 888;
- block2_2_.sourceSSRC = kSsrc2;
+ block2_2_.source_ssrc = kSsrc2;
ssrc1block1_.push_back(block1_1_);
ssrc1block2_.push_back(block1_2_);
@@ -57,9 +57,10 @@ class ReportBlockStatsTest : public ::testing::Test {
RtcpStatistics RtcpReportBlockToRtcpStatistics(
const RTCPReportBlock& stats) {
RtcpStatistics block;
- block.packets_lost = stats.cumulativeLost;
- block.fraction_lost = stats.fractionLost;
- block.extended_highest_sequence_number = stats.extendedHighSeqNum;
+ block.packets_lost = stats.packets_lost;
+ block.fraction_lost = stats.fraction_lost;
+ block.extended_highest_sequence_number =
+ stats.extended_highest_sequence_number;
block.jitter = stats.jitter;
return block;
}
@@ -81,26 +82,26 @@ TEST_F(ReportBlockStatsTest, AggregateAndStore_NoSsrc) {
ReportBlockStats stats;
std::vector<RTCPReportBlock> empty;
RTCPReportBlock aggregated = stats.AggregateAndStore(empty);
- EXPECT_EQ(0U, aggregated.fractionLost);
- EXPECT_EQ(0U, aggregated.cumulativeLost);
+ EXPECT_EQ(0U, aggregated.fraction_lost);
+ EXPECT_EQ(0U, aggregated.packets_lost);
EXPECT_EQ(0U, aggregated.jitter);
- EXPECT_EQ(0U, aggregated.extendedHighSeqNum);
+ EXPECT_EQ(0U, aggregated.extended_highest_sequence_number);
}
TEST_F(ReportBlockStatsTest, AggregateAndStore_OneSsrc) {
ReportBlockStats stats;
RTCPReportBlock aggregated = stats.AggregateAndStore(ssrc1block1_);
// One ssrc, no aggregation done.
- EXPECT_EQ(123U, aggregated.fractionLost);
- EXPECT_EQ(10U, aggregated.cumulativeLost);
+ EXPECT_EQ(123U, aggregated.fraction_lost);
+ EXPECT_EQ(10U, aggregated.packets_lost);
EXPECT_EQ(777U, aggregated.jitter);
- EXPECT_EQ(24000U, aggregated.extendedHighSeqNum);
+ EXPECT_EQ(24000U, aggregated.extended_highest_sequence_number);
aggregated = stats.AggregateAndStore(ssrc1block2_);
- EXPECT_EQ(0U, aggregated.fractionLost);
- EXPECT_EQ(15U, aggregated.cumulativeLost);
+ EXPECT_EQ(0U, aggregated.fraction_lost);
+ EXPECT_EQ(15U, aggregated.packets_lost);
EXPECT_EQ(222U, aggregated.jitter);
- EXPECT_EQ(24100U, aggregated.extendedHighSeqNum);
+ EXPECT_EQ(24100U, aggregated.extended_highest_sequence_number);
// fl: 100 * (15-10) / (24100-24000) = 5%
EXPECT_EQ(5, stats.FractionLostInPercent());
@@ -109,17 +110,17 @@ TEST_F(ReportBlockStatsTest, AggregateAndStore_OneSsrc) {
TEST_F(ReportBlockStatsTest, AggregateAndStore_TwoSsrcs) {
ReportBlockStats stats;
RTCPReportBlock aggregated = stats.AggregateAndStore(ssrc12block1_);
- EXPECT_EQ(0U, aggregated.fractionLost);
- EXPECT_EQ(10U + 111U, aggregated.cumulativeLost);
+ EXPECT_EQ(0U, aggregated.fraction_lost);
+ EXPECT_EQ(10U + 111U, aggregated.packets_lost);
EXPECT_EQ((777U + 555U) / 2, aggregated.jitter);
- EXPECT_EQ(0U, aggregated.extendedHighSeqNum);
+ EXPECT_EQ(0U, aggregated.extended_highest_sequence_number);
aggregated = stats.AggregateAndStore(ssrc12block2_);
// fl: 255 * ((15-10) + (136-111)) / ((24100-24000) + (8800-8500)) = 19
- EXPECT_EQ(19U, aggregated.fractionLost);
- EXPECT_EQ(15U + 136U, aggregated.cumulativeLost);
+ EXPECT_EQ(19U, aggregated.fraction_lost);
+ EXPECT_EQ(15U + 136U, aggregated.packets_lost);
EXPECT_EQ((222U + 888U) / 2, aggregated.jitter);
- EXPECT_EQ(0U, aggregated.extendedHighSeqNum);
+ EXPECT_EQ(0U, aggregated.extended_highest_sequence_number);
// fl: 100 * ((15-10) + (136-111)) / ((24100-24000) + (8800-8500)) = 7%
EXPECT_EQ(7, stats.FractionLostInPercent());

Powered by Google App Engine
This is Rietveld 408576698