diff options
| author | wtc@google.com <wtc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-14 00:22:51 +0000 |
|---|---|---|
| committer | wtc@google.com <wtc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-14 00:22:51 +0000 |
| commit | a4fb51b3a5e6c541d1d840c32c57dbf1c09345c0 (patch) | |
| tree | fc09612fc8cb675b9b11098e879796dbdc578f09 /net/http | |
| parent | a95b6ebeb94363671433029e97a488bccb17ff17 (diff) | |
| download | chromium_src-a4fb51b3a5e6c541d1d840c32c57dbf1c09345c0.zip chromium_src-a4fb51b3a5e6c541d1d840c32c57dbf1c09345c0.tar.gz chromium_src-a4fb51b3a5e6c541d1d840c32c57dbf1c09345c0.tar.bz2 | |
A minor tweak of r5245. I'm more confident that WinHTTP
knows a transaction is complete when WinHttpReadData returns
a zero byte count. So it's safer to NOT notify the request
throttle when WinHttpReadData returns a nonzero byte count,
even if we have read the number of bytes specified in the
Content-Length response header. I verified that this is
still enough to fix the bug (unresponsiveness).
R=darin
BUG=4302
Review URL: http://codereview.chromium.org/10867
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5415 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http')
| -rw-r--r-- | net/http/http_transaction_winhttp.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/http/http_transaction_winhttp.cc b/net/http/http_transaction_winhttp.cc index 727bbe3..460ad8a 100644 --- a/net/http/http_transaction_winhttp.cc +++ b/net/http/http_transaction_winhttp.cc @@ -1437,7 +1437,7 @@ int HttpTransactionWinHttp::DidReadData(DWORD num_bytes) { // We have read the entire response. Mark the request done to unblock a // queued request. - if (rv == 0 || content_length_remaining_ == 0) { + if (rv == 0) { DCHECK(request_submitted_); request_submitted_ = false; session_->request_throttle()->NotifyRequestDone(connect_peer_); |