Don't consider '\0' as whitespace when parsing HTTP headers.

net/http/http_util.cc

 // Copyright (c) 2012 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.
 
 // The rules for parsing content-types were borrowed from Firefox:
 // http://lxr.mozilla.org/mozilla/source/netwerk/base/src/nsURLHelper.cpp#834
 
 #include "net/http/http_util.h"
 
 #include <algorithm>
@@ skipping 408 matching lines @@
   for (const char* header : kNonCoalescingHeaders) {
     if (base::LowerCaseEqualsASCII(base::StringPiece(name_begin, name_end),
                                    header)) {
       return true;
     }
   }
   return false;
 }
 
 bool HttpUtil::IsLWS(char c) {
-  return strchr(HTTP_LWS, c) != NULL;
+  base::StringPiece kLws(HTTP_LWS);

[mmenke, 2016/12/06 19:08:24]

Needs a const.  Also, LWS is one thing, but kLws seems a bit much. 
kWhiteSpaceCharacters?  k<anything with a vowel>?

[eroman, 2016/12/06 19:19:34]

Done.

+  return kLws.find(c) != base::StringPiece::npos;
 }
 
 // static
 void HttpUtil::TrimLWS(std::string::const_iterator* begin,
                        std::string::const_iterator* end) {
   TrimLWSImplementation(begin, end);
 }
 
 // static
 base::StringPiece HttpUtil::TrimLWS(const base::StringPiece& string) {
@@ skipping 633 matching lines @@
   return true;
 }
 
 bool HttpUtil::NameValuePairsIterator::IsQuote(char c) const {
   if (strict_quotes_)
     return c == '"';
   return HttpUtil::IsQuote(c);
 }
 
 }  // namespace net