From 42636e82d0570d7f32d7df2fc8af54ab28b0640f Mon Sep 17 00:00:00 2001 From: "pwestin@webrtc.org" Date: Thu, 2 May 2013 21:02:04 +0000 Subject: [PATCH] Removing bad code resulting in flaky test. BUG=1723 R=niklas.enbom@webrtc.org Review URL: https://webrtc-codereview.appspot.com/1390004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@3941 4adac7df-926f-26a2-2b94-8c16560cd09d --- .../test/auto_test/source/vie_autotest_rtp_rtcp.cc | 9 ++++++--- 1 file changed, 6 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 43b9021515..6ec57dfd99 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 @@ -431,9 +431,12 @@ void ViEAutoTest::ViERtpRtcpStandardTest() EXPECT_EQ(0, ViE.rtp_rtcp->GetBandwidthUsage( tbChannel.videoChannel, sentTotalBitrate, sentVideoBitrate, sentFecBitrate, sentNackBitrate)); - if (FLAGS_include_timing_dependent_tests) { - EXPECT_EQ(sentTotalBitrate, 0u); - } +// TODO(pwestin): This is incorrect due to the fact that we measure the bitrate +// before the pacer and not after. +// +// if (FLAGS_include_timing_dependent_tests) { +// EXPECT_EQ(sentTotalBitrate, 0u); +// } // Network reception back. Video should now be sent. ViETest::Log("Network Up...\n"); ViE.network->SetNetworkTransmissionState(tbChannel.videoChannel, true);