From f24c4a3b8d025138382dadbc06591cb686e23b23 Mon Sep 17 00:00:00 2001 From: "stefan@webrtc.org" Date: Wed, 23 Jul 2014 10:27:41 +0000 Subject: [PATCH] Fix flaky ramp-up test. Don't require the first estimate to be less than the target bitrate. There are other tests verifying that BWE works, so it's enough for this test to measure the time it takes to ramp-up. R=mflodman@webrtc.org Review URL: https://webrtc-codereview.appspot.com/20979004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@6764 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/video/rampup_tests.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/webrtc/video/rampup_tests.cc b/webrtc/video/rampup_tests.cc index af3be8610b..e1dd95a67f 100644 --- a/webrtc/video/rampup_tests.cc +++ b/webrtc/video/rampup_tests.cc @@ -99,7 +99,6 @@ void StreamObserver::OnReceiveBitrateChanged( // start bitrate, but due to the BWE implementation we can't guarantee the // first estimate really is as high as the start bitrate. EXPECT_GT(bitrate, 0.9 * start_bitrate_bps_); - EXPECT_LT(bitrate, expected_bitrate_bps_); start_bitrate_bps_ = 0; } if (bitrate >= expected_bitrate_bps_) {