diff --git a/webrtc/video_engine/internal/video_send_stream.cc b/webrtc/video_engine/internal/video_send_stream.cc index a0a99e076f..55921c2ca7 100644 --- a/webrtc/video_engine/internal/video_send_stream.cc +++ b/webrtc/video_engine/internal/video_send_stream.cc @@ -166,8 +166,8 @@ VideoSendStream::VideoSendStream(newapi::Transport* transport, assert(network_ != NULL); network_->RegisterSendTransport(channel_, transport_adapter_); - network_->SetMTU(channel_, config_.rtp.max_packet_size + - VideoSendStream::Config::kDefaultPacketOverheader); + // 28 to match packet overhead in ModuleRtpRtcpImpl. + network_->SetMTU(channel_, config_.rtp.max_packet_size + 28); if (config.encoder) { external_codec_ = ViEExternalCodec::GetInterface(video_engine); diff --git a/webrtc/video_engine/new_include/video_send_stream.h b/webrtc/video_engine/new_include/video_send_stream.h index 9ae78bd40c..5c0ec32738 100644 --- a/webrtc/video_engine/new_include/video_send_stream.h +++ b/webrtc/video_engine/new_include/video_send_stream.h @@ -84,8 +84,7 @@ class VideoSendStream { start_state(NULL) {} VideoCodec codec; - static const size_t kDefaultPacketOverheader = 20 + 20; // IPv4 + TCP - static const size_t kDefaultMaxPacketSize = 1500 - kDefaultPacketOverheader; + static const size_t kDefaultMaxPacketSize = 1500 - 40; // TCP over IPv4. struct Rtp { Rtp() : mode(newapi::kRtcpReducedSize), max_packet_size(kDefaultMaxPacketSize) {} diff --git a/webrtc/video_engine/test/send_stream_tests.cc b/webrtc/video_engine/test/send_stream_tests.cc index 1c9f32d9ae..69923a704a 100644 --- a/webrtc/video_engine/test/send_stream_tests.cc +++ b/webrtc/video_engine/test/send_stream_tests.cc @@ -443,7 +443,7 @@ TEST_F(VideoSendStreamTest, MaxPacketSize) { EXPECT_TRUE( rtp_header_parser_->Parse(packet, static_cast(length), &header)); - EXPECT_TRUE(length <= max_length_); + EXPECT_LE(length, max_length_); accumulated_size_ += length;