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

Issue 165365: Merge 22100 - Fix 307 redirects of a POST request to actually send the body a... (Closed)

Created:
11 years, 4 months ago by laforge
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com, darin (slow to review), willchan no longer on Chromium
Visibility:
Public.

Description

Merge 22100 - Fix 307 redirects of a POST request to actually send the body again, and add a test. Note: the fix is trivial, but adding support for a test required teaching the FileHandler of testserver to consume a post body. BUG=16832 TEST=covered by unit test R=abarth Review URL: http://codereview.chromium.org/149710 TBR=darin@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=23141

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -4 lines) Patch
A + net/data/url_request_unittest/redirect307-to-echo View 0 chunks +-1 lines, --1 lines 0 comments Download
A + net/data/url_request_unittest/redirect307-to-echo.mock-http-headers View 0 chunks +-1 lines, --1 lines 0 comments Download
D net/data/url_request_unittest/redirect307-to-echoall View 1 chunk +0 lines, -1 line 0 comments Download
D net/data/url_request_unittest/redirect307-to-echoall.mock-http-headers View 1 chunk +0 lines, -2 lines 0 comments Download
MM net/url_request/url_request.cc View 1 chunk +1 line, -1 line 0 comments Download
MM net/url_request/url_request_unittest.cc View 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
laforge
11 years, 4 months ago (2009-08-12 02:21:48 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698