admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Try to deflake VideoSendStream tests with FlexFEC.

Browse Source 
BUG=webrtc:6744
NOTRY=True # goma doesn't work on android_more_configs bot

Review-Url: https://codereview.webrtc.org/2523993002
Cr-Commit-Position: refs/heads/master@{#15208}
This commit is contained in:
brandtr 2016-11-23 04:42:26 -08:00 committed by Commit bot
parent 0adb8285b1
commit 0c5a154075
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
webrtc/video/video_send_stream_tests.cc
View File

@ -546,7 +546,6 @@ class FlexfecObserver : public test::EndToEndTest {
: EndToEndTest(VideoSendStreamTest::kDefaultTimeoutMs),
payload_name_(codec),
use_nack_(use_nack),
send_count_(0),
sent_media_(false),
sent_flexfec_(false),
header_extensions_enabled_(header_extensions_enabled) {
@ -568,7 +567,6 @@ class FlexfecObserver : public test::EndToEndTest {
RTPHeader header;
EXPECT_TRUE(parser_->Parse(packet, length, &header));
++send_count_;
if (header.payloadType == VideoSendStreamTest::kFlexfecPayloadType) {
EXPECT_EQ(VideoSendStreamTest::kFlexfecSendSsrc, header.ssrc);
sent_flexfec_ = true;
@ -585,7 +583,7 @@ class FlexfecObserver : public test::EndToEndTest {
EXPECT_TRUE(header.extension.hasTransportSequenceNumber);
}
if (send_count_ > 100 && sent_media_ && sent_flexfec_) {
if (sent_media_ && sent_flexfec_) {
observation_complete_.Set();
}
@ -637,7 +635,6 @@ class FlexfecObserver : public test::EndToEndTest {
std::unique_ptr<VideoEncoder> encoder_;
const std::string payload_name_;
const bool use_nack_;
int send_count_;
bool sent_media_;
bool sent_flexfec_;
bool header_extensions_enabled_;