net/http: drop unused "broken" field from persistConn

Change-Id: Ic65cf98c090c73299b5e88e642e91139315d8e52
Reviewed-on: https://go-review.googlesource.com/c/go/+/722221
Auto-Submit: Damien Neil <dneil@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Nicholas Husin <husin@google.com>
Reviewed-by: Nicholas Husin <nsh@golang.org>

diff --git a/src/net/http/transport.go b/src/net/http/transport.go
index a560765d331d6503ad6c78fbccfaf89f69a652bf..4e6b07f34d61ada316de683b5e4af469da52e018 100644 (file)
--- a/src/net/http/transport.go
+++ b/src/net/http/transport.go
@@ -2110,7 +2110,6 @@ type persistConn struct {
        numExpectedResponses int
        closed               error // set non-nil when conn is closed, before closech is closed
        canceledErr          error // set non-nil if conn is canceled
-       broken               bool  // an error has happened on this connection; marked broken so it's not reused.
        reused               bool  // whether conn has had successful request/response and is being reused.
        // mutateHeaderFunc is an optional func to modify extra
        // headers on each outbound request before it's written. (the
@@ -2925,7 +2924,6 @@ func (pc *persistConn) closeLocked(err error) {
        if err == nil {
                panic("nil error")
        }
-       pc.broken = true
        if pc.closed == nil {
                pc.closed = err
                pc.t.decConnsPerHost(pc.cacheKey)