From 5c928ebd1d5d61deca0b5458e31005ac5dc1c31f Mon Sep 17 00:00:00 2001 From: "asapersson@webrtc.org" Date: Wed, 25 Feb 2015 11:47:11 +0000 Subject: [PATCH] Let first packet through to avoid getting key frame requests (and no nacks) for EndToEndTest.ReceivedFecPacketsNotNacked. BUG=4328 R=pbos@webrtc.org Review URL: https://webrtc-codereview.appspot.com/38259004 Cr-Commit-Position: refs/heads/master@{#8502} git-svn-id: http://webrtc.googlecode.com/svn/trunk@8502 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/video/end_to_end_tests.cc | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc index 66cd74f4e..3d46cc344 100644 --- a/webrtc/video/end_to_end_tests.cc +++ b/webrtc/video/end_to_end_tests.cc @@ -565,7 +565,7 @@ void EndToEndTest::TestReceivedFecPacketsNotNacked( public: explicit FecNackObserver(const FakeNetworkPipe::Config& config) : EndToEndTest(kDefaultTimeoutMs, config), - state_(kDropEveryOtherPacketUntilFec), + state_(kFirstPacket), fec_sequence_number_(0), has_last_sequence_number_(false), last_sequence_number_(0) {} @@ -592,6 +592,9 @@ void EndToEndTest::TestReceivedFecPacketsNotNacked( bool fec_packet = encapsulated_payload_type == kUlpfecPayloadType; switch (state_) { + case kFirstPacket: + state_ = kDropEveryOtherPacketUntilFec; + break; case kDropEveryOtherPacketUntilFec: if (fec_packet) { state_ = kDropAllMediaPacketsUntilFec; @@ -649,6 +652,7 @@ void EndToEndTest::TestReceivedFecPacketsNotNacked( } enum { + kFirstPacket, kDropEveryOtherPacketUntilFec, kDropAllMediaPacketsUntilFec, kVerifyFecPacketNotInNackList,