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

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

Issue 2437503004: Set actual transport overhead in rtp_rtcp (Closed)
Patch Set: Created 4 years, 2 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.cc
diff --git a/webrtc/modules/rtp_rtcp/test/testAPI/test_api.cc b/webrtc/modules/rtp_rtcp/test/testAPI/test_api.cc
index d1e520de8fc8026db6beab86e6aa169676b0c901..e0dac95b400a5be1ec877e493552927dbd3ea067 100644
--- a/webrtc/modules/rtp_rtcp/test/testAPI/test_api.cc
+++ b/webrtc/modules/rtp_rtcp/test/testAPI/test_api.cc
@@ -135,11 +135,11 @@ TEST_F(RtpRtcpAPITest, MTU) {
EXPECT_EQ(0, module_->SetMaxTransferUnit(1234));
EXPECT_EQ(1234 - 20 - 8, module_->MaxPayloadLength());
- EXPECT_EQ(0, module_->SetTransportOverhead(true, true, 12));
- EXPECT_EQ(1234 - 20 - 20 - 20 - 12, module_->MaxPayloadLength());
+ EXPECT_EQ(0, module_->SetTransportOverhead(28));
+ EXPECT_EQ(1234 - 28, module_->MaxPayloadLength());
- EXPECT_EQ(0, module_->SetTransportOverhead(false, false, 0));
- EXPECT_EQ(1234 - 20 - 8, module_->MaxPayloadLength());
+ EXPECT_EQ(0, module_->SetTransportOverhead(44));
+ EXPECT_EQ(1234 - 44, module_->MaxPayloadLength());
}
TEST_F(RtpRtcpAPITest, SSRC) {

Powered by Google App Engine
This is Rietveld 408576698