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

Issue 2816583002: Pass the list of packets expected to be lost directly into VerifyLoss() as a C++ vector, as opposed… (Closed)

Created:
3 years, 8 months ago by danzh1
Modified:
3 years, 8 months ago
Reviewers:
Jana
CC:
Victor Vasiliev, chromium-reviews, cbentzel+watch_chromium.org, net-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Pass the list of packets expected to be lost directly into VerifyLoss() as a C++ vector, as opposed to a C array. n/a (test-only change) Merge internal change: 152503943 R=jri@chromium.org BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -54 lines) Patch
M net/quic/core/congestion_control/general_loss_algorithm_test.cc View 19 chunks +38 lines, -54 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 7 (5 generated)
danzh1
3 years, 8 months ago (2017-04-11 17:32:45 UTC) #2
Jana
3 years, 8 months ago (2017-04-11 17:55:17 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698