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

Issue 2560603003: Test for all our CC algorithms under >1 network scenarios. (Closed)

Created:
4 years ago by ianswett
Modified:
4 years ago
Reviewers:
Ryan Hamilton
CC:
chromium-reviews, cbentzel+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Test for all our CC algorithms under >1 network scenarios. The purpose of this test suite is to exercise all our existing congestion-control algorithms under a variety of network scenarios, using our handy-dandy network simulator. Right now, the tests are parameterized by congestion-control algorithm. Each test performs a simple transfers under a different network condition, using the given CC algorithm. It measures the amount of time it takes to perform the transfer and prints out some stats. If the transfer does not complete within a reasonable amount of time (where "reasonable" depends on the scenario), the test will fail. The tests may use additional pass/fail criteria, such as maximum acceptable retransmissions, in the future. I've included a few simple scenarios to get started. I expect these to expand as we discover new conditions to test. Merge internal change: 140494757 R=rch@chromium.org BUG=

Patch Set 1 #

Patch Set 2 : Add license. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+360 lines, -0 lines) Patch
M net/net.gypi View 1 chunk +1 line, -0 lines 0 comments Download
A net/quic/core/congestion_control/send_algorithm_test.cc View 1 1 chunk +359 lines, -0 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 2 (0 generated)
ianswett
4 years ago (2016-12-06 21:40:08 UTC) #1
Ryan Hamilton
4 years ago (2016-12-06 22:16:30 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698