[new http] Normalize line continuations in response headers.

net/http/http_response_headers_unittest.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/pickle.h"
 #include "base/time.h"
 #include "net/http/http_response_headers.h"
@@ skipping 50 matching lines @@
 
   EXPECT_EQ(test.expected_response_code, parsed->response_code());
 }
 
 } // end namespace
 
 // Check that we normalize headers properly.
 TEST(HttpResponseHeadersTest, NormalizeHeadersWhitespace) {
   TestData test = {
     "HTTP/1.1    202   Accepted  \n"
-    "  Content-TYPE  : text/html; charset=utf-8  \n"
+    "Content-TYPE  : text/html; charset=utf-8  \n"
     "Set-Cookie: a \n"
     "Set-Cookie:   b \n",
 
     "HTTP/1.1 202 Accepted\n"
     "Content-TYPE: text/html; charset=utf-8\n"
     "Set-Cookie: a, b\n",
 
     202
   };
   TestCommon(test);
 }
 
+// Check that we normalize headers properly (header name is invalid if starts
+// with LWS).
+TEST(HttpResponseHeadersTest, NormalizeHeadersLeadingWhitespace) {
+  TestData test = {
+    "HTTP/1.1    202   Accepted  \n"
+    // Starts with space -- will be skipped as invalid.
+    "  Content-TYPE  : text/html; charset=utf-8  \n"
+    "Set-Cookie: a \n"
+    "Set-Cookie:   b \n",
+
+    "HTTP/1.1 202 Accepted\n"
+    "Set-Cookie: a, b\n",
+
+    202
+  };
+  TestCommon(test);
+}
+
 TEST(HttpResponseHeadersTest, BlankHeaders) {
   TestData test = {
     "HTTP/1.1 200 OK\n"
     "Header1 :          \n"
     "Header2: \n"
     "Header3:\n"
     "Header4\n"
     "Header5    :\n",
 
     "HTTP/1.1 200 OK\n"
@@ skipping 878 matching lines @@
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) {
     string headers(tests[i].headers);
     HeadersToRaw(&headers);
     scoped_refptr<HttpResponseHeaders> parsed =
         new HttpResponseHeaders(headers);
 
     EXPECT_EQ(tests[i].expected_keep_alive, parsed->IsKeepAlive());
   }
 }