Mark QUIC broken when the network blackholes after the handshake

net/http/http_network_transaction.cc

Index: net/http/http_network_transaction.cc
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index e0105c60396fe3f891916b12114cff7c14da899b..85f54cd7d3ca457880fcdf34280366510c4a35ea 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -1533,6 +1533,13 @@ int HttpNetworkTransaction::HandleIOError(int error) {
         error = OK;
       }
       break;
+    case ERR_QUIC_BROKEN_ERROR:
+      DCHECK(GetResponseHeaders() == nullptr);

[Jana, 2017/04/06 02:29:18]

why is this a DCHECK and not if (ShouldResendRequest())?

[Ryan Hamilton, 2017/04/06 03:43:38]

Because this error code is never returned when headers have been received. And
I'm not checking ShouldResendRequest (which an earlier patch set did) because it
turns out that HandleIOError is not called after headers have been received, so
it's rather redundant.

+      net_log_.AddEventWithNetErrorCode(
+          NetLogEventType::HTTP_TRANSACTION_RESTART_AFTER_ERROR, error);
+      ResetConnectionAndRequestForResend();
+      error = OK;
+      break;
     case ERR_SPDY_PING_FAILED:
     case ERR_SPDY_SERVER_REFUSED_STREAM:
     case ERR_QUIC_HANDSHAKE_FAILED: