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

Issue 9383029: Revert 121572 - Add client for background testing of HTTP pipelining. (Closed)

Created:
8 years, 10 months ago by James Simonsen
Modified:
8 years, 10 months ago
Reviewers:
James Simonsen
CC:
chromium-reviews, darin-cc_chromium.org, cbentzel+watch_chromium.org, pam+watch_chromium.org
Visibility:
Public.

Description

Revert 121572 - Add client for background testing of HTTP pipelining. Note this isn't hooked up, so it won't affect users yet. Once the servers are up, it'll be hooked up to a field test, much like the UDP echo test. Also missing is a force-pipelining option for requests. I'll add that in a separate CL. BUG=110794 TEST=unit_tests Review URL: http://codereview.chromium.org/9302024 TBR=mmenke@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=121605

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -777 lines) Patch
D chrome/browser/net/http_pipelining_compatibility_client.h View 1 chunk +0 lines, -125 lines 0 comments Download
D chrome/browser/net/http_pipelining_compatibility_client.cc View 1 chunk +0 lines, -188 lines 0 comments Download
D chrome/browser/net/http_pipelining_compatibility_client_unittest.cc View 1 chunk +0 lines, -440 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 chunk +0 lines, -1 line 0 comments Download
D chrome/test/data/http_pipelining/alphabet.txt View 1 chunk +0 lines, -1 line 0 comments Download
D chrome/test/data/http_pipelining/alphabet.txt.mock-http-headers View 1 chunk +0 lines, -1 line 0 comments Download
M net/base/net_error_list.h View 3 chunks +4 lines, -4 lines 0 comments Download
M net/tools/testserver/testserver.py View 5 chunks +1 line, -15 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
James Simonsen
8 years, 10 months ago (2012-02-11 01:38:34 UTC) #1
mmenke
8 years, 10 months ago (2012-02-11 01:42:35 UTC) #2
LGTM, thanks.

Powered by Google App Engine
This is Rietveld 408576698