From f31dc860b52665646946d986146fcdf7d2e60b36 Mon Sep 17 00:00:00 2001 From: bnc Date: Mon, 1 Dec 2014 05:47:16 -0800 Subject: 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} --- net/http/http_network_transaction.cc | 5 ----- 1 file changed, 5 deletions(-) (limited to 'net/http/http_network_transaction.cc') 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. -- cgit v1.1