diff options
| author | zea@chromium.org <zea@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-22 00:02:00 +0000 |
|---|---|---|
| committer | zea@chromium.org <zea@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-22 00:02:00 +0000 |
| commit | 36c9ee8479a7105eb5ddfa4ee499b003018b3079 (patch) | |
| tree | b07d32ccb782495e745744abc19870269ed83ef3 /testing/android | |
| parent | bacbac5abff91c6c00dffeaba381a69e2b106ee8 (diff) | |
| download | chromium_src-36c9ee8479a7105eb5ddfa4ee499b003018b3079.zip chromium_src-36c9ee8479a7105eb5ddfa4ee499b003018b3079.tar.gz chromium_src-36c9ee8479a7105eb5ddfa4ee499b003018b3079.tar.bz2 | |
Revert 138168 - net: Fix a regression that broke file uploading with http authentication
Failing net_unit_tests RequestSocketsMultipleTimesDoesNothing.
When uploading a file with http authentication, UploadData is reused
for a new UploadDataStream. Hence, we should close the files in UploadData
if already opened and read, so we can reread the files from the beginning.
The regression was caused by r123677. Special thanks to thomas.themel for
identifying the offending patch and helping us to fix the issue.
BUG=128574
TEST=added a unit test; manually confirmed that uploading a file with basic authentication, and multi-round authentication with NTLM and Negotiate worked.
Review URL: https://chromiumcodereview.appspot.com/10408042
TBR=satorux@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10388221
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138175 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'testing/android')
0 files changed, 0 insertions, 0 deletions