Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(57)

Side by Side Diff: webrtc/modules/congestion_controller/delay_based_bwe_unittest.cc

Issue 2512693002: Implement Theil-Sen's method for fitting a line to noisy data (used in bandwidth estimation). (Closed)
Patch Set: Use PercentileFilter to find median slope Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 211 matching lines...) Expand 10 before | Expand all | Expand 10 after
222 } 222 }
223 223
224 TEST_F(DelayBasedBweTrendlineExperimentTest, CapacityDropNegOffsetChange) { 224 TEST_F(DelayBasedBweTrendlineExperimentTest, CapacityDropNegOffsetChange) {
225 CapacityDropTestHelper(1, false, 1267, -30000); 225 CapacityDropTestHelper(1, false, 1267, -30000);
226 } 226 }
227 227
228 TEST_F(DelayBasedBweTrendlineExperimentTest, CapacityDropOneStreamWrap) { 228 TEST_F(DelayBasedBweTrendlineExperimentTest, CapacityDropOneStreamWrap) {
229 CapacityDropTestHelper(1, true, 600, 0); 229 CapacityDropTestHelper(1, true, 600, 0);
230 } 230 }
231 231
232 class DelayBasedBweTheilSenExperimentTest : public DelayBasedBweTest {
233 public:
234 DelayBasedBweTheilSenExperimentTest()
235 : override_field_trials_("WebRTC-BweTheilSenFilter/Enabled-20,4/") {}
236
237 protected:
238 void SetUp() override {
239 bitrate_estimator_.reset(new DelayBasedBwe(&clock_));
brandtr 2016/11/28 16:27:19 I think you could move this to the constructor?
terelius 2016/12/02 16:45:52 Done.
240 }
241
242 test::ScopedFieldTrials override_field_trials_;
243 };
244
245 TEST_F(DelayBasedBweTheilSenExperimentTest, RateIncreaseRtpTimestamps) {
246 RateIncreaseRtpTimestampsTestHelper(1240);
247 }
248
249 TEST_F(DelayBasedBweTheilSenExperimentTest, CapacityDropOneStream) {
250 CapacityDropTestHelper(1, false, 600, 0);
251 }
252
253 TEST_F(DelayBasedBweTheilSenExperimentTest, CapacityDropPosOffsetChange) {
254 CapacityDropTestHelper(1, false, 600, 30000);
255 }
256
257 TEST_F(DelayBasedBweTheilSenExperimentTest, CapacityDropNegOffsetChange) {
258 CapacityDropTestHelper(1, false, 1267, -30000);
259 }
260
261 TEST_F(DelayBasedBweTheilSenExperimentTest, CapacityDropOneStreamWrap) {
262 CapacityDropTestHelper(1, true, 600, 0);
263 }
264
232 } // namespace webrtc 265 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698