diff options
author | bnc <bnc@chromium.org> | 2014-12-01 05:47:16 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-12-01 13:47:42 +0000 |
commit | f31dc860b52665646946d986146fcdf7d2e60b36 (patch) | |
tree | 7648356b983fe586255cfdcf784546df113ea50f /net/http/http_network_transaction.cc | |
parent | feaaf10864e44966e1a2f4dfe5298f3cbf1e1934 (diff) | |
download | chromium_src-f31dc860b52665646946d986146fcdf7d2e60b36.zip chromium_src-f31dc860b52665646946d986146fcdf7d2e60b36.tar.gz chromium_src-f31dc860b52665646946d986146fcdf7d2e60b36.tar.bz2 |
Remove redundant ERR_QUIC_HANDSHAKE_FAILED handling.
DoReadHeadersComplete handles |result == ERR_QUIC_HANDSHAKE_FAILED|, but a few
lines later, calls |HandleIOError|, which does the same. This is redundant,
therefore I am removing the first check.
Please note that this results in two things in case of an
|ERR_QUIC_HANDSHAKE_FAILED|: |HandleClientAuthError| will be called, and
AddEventWithNetErrorCode will be executed.
BUG=
Review URL: https://codereview.chromium.org/747773004
Cr-Commit-Position: refs/heads/master@{#306186}
Diffstat (limited to 'net/http/http_network_transaction.cc')
-rw-r--r-- | net/http/http_network_transaction.cc | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc index 8cddaf5..b4ef356 100644 --- a/net/http/http_network_transaction.cc +++ b/net/http/http_network_transaction.cc @@ -961,11 +961,6 @@ int HttpNetworkTransaction::DoReadHeadersComplete(int result) { return result; } - if (result == ERR_QUIC_HANDSHAKE_FAILED) { - ResetConnectionAndRequestForResend(); - return OK; - } - // ERR_CONNECTION_CLOSED is treated differently at this point; if partial // response headers were received, we do the best we can to make sense of it // and send it back up the stack. |