Clean up HttpResponseHeaders::IsKeepAlive. Use the...

net/http/http_response_headers_unittest.cc

Index: net/http/http_response_headers_unittest.cc
===================================================================
--- net/http/http_response_headers_unittest.cc	(revision 11044)
+++ net/http/http_response_headers_unittest.cc	(working copy)
@@ -1042,6 +1042,17 @@
     const char* headers;
     bool expected_keep_alive;
   } tests[] = {
+    // The status line fabricated by HttpNetworkTransaction for a 0.9 response.
+    // Treated as 0.9.
+    { "HTTP/0.9 200 OK",
+      false
+    },
+    // This could come from a broken server.  Treated as 1.0 because it has a
+    // header.
+    { "HTTP/0.9 200 OK\n"
+      "connection: keep-alive\n",
+      true
+    },
     { "HTTP/1.1 200 OK\n",
       true
     },
@@ -1072,10 +1083,22 @@
       "connection: keep-alive\n",
       true
     },
+    { "HTTP/1.0 200 OK\n"
+      "proxy-connection: close\n",
+      false
+    },
+    { "HTTP/1.0 200 OK\n"
+      "proxy-connection: keep-alive\n",
+      true
+    },
     { "HTTP/1.1 200 OK\n"
       "proxy-connection: close\n",
       false
     },
+    { "HTTP/1.1 200 OK\n"
+      "proxy-connection: keep-alive\n",
+      true
+    },
   };
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) {
     string headers(tests[i].headers);