From cdd3d4d139b4e6d9eb3cc43a76d3d323e90f1d04 Mon Sep 17 00:00:00 2001 From: "stefan@webrtc.org" Date: Mon, 23 Sep 2013 09:43:07 +0000 Subject: [PATCH] Revert test change in r4808. This was supposed to be an EXPECT_GT, I just misunderstood it in the previous CL. Added a sleep after the EXPECT_GT and before bytes_received_after = bytes_received_before. BUG=1790 R=mflodman@webrtc.org Review URL: https://webrtc-codereview.appspot.com/2265006 git-svn-id: http://webrtc.googlecode.com/svn/trunk@4809 4adac7df-926f-26a2-2b94-8c16560cd09d --- .../test/auto_test/source/vie_autotest_rtp_rtcp.cc | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/webrtc/video_engine/test/auto_test/source/vie_autotest_rtp_rtcp.cc b/webrtc/video_engine/test/auto_test/source/vie_autotest_rtp_rtcp.cc index e78e80ee6..3b4753eca 100644 --- a/webrtc/video_engine/test/auto_test/source/vie_autotest_rtp_rtcp.cc +++ b/webrtc/video_engine/test/auto_test/source/vie_autotest_rtp_rtcp.cc @@ -561,16 +561,15 @@ void ViEAutoTest::ViERtpRtcpStandardTest() // Allow the encoder to finish the current frame before we expect that no // additional packets will be sent. AutoTestSleep(kAutoTestSleepTimeMs); - bytes_received_after = bytes_received_before; - AutoTestSleep(kAutoTestSleepTimeMs); EXPECT_EQ(0, ViE.rtp_rtcp->GetRTPStatistics(tbChannel.videoChannel, bytes_sent_before, packets_sent_before, bytes_received_before, packets_received_before)); if (FLAGS_include_timing_dependent_tests) { - EXPECT_EQ(bytes_received_before, bytes_received_after); + EXPECT_GT(bytes_received_before, bytes_received_after); } + bytes_received_after = bytes_received_before; AutoTestSleep(kAutoTestSleepTimeMs); EXPECT_EQ(0, ViE.rtp_rtcp->GetRTPStatistics(tbChannel.videoChannel, bytes_sent_after,