Add UMAs for checking header values against RFC 7230 in //net

net/http/http_util.h

 // 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.
 
 #ifndef NET_HTTP_HTTP_UTIL_H_
 #define NET_HTTP_HTTP_UTIL_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 62 matching lines @@
   // named |name|.
   static bool IsSafeHeader(const std::string& name);
 
   // Returns true if |name| is a valid HTTP header name.
   static bool IsValidHeaderName(const std::string& name);
 
   // Returns false if |value| contains NUL or CRLF. This method does not perform
   // a fully RFC-2616-compliant header value validation.
   static bool IsValidHeaderValue(const std::string& value);
 
+  // Returns true if the string is a valid HTTP header value according to
+  // RFC 7230 and doesn't contain CR or LF.
+  // i.e. returns true if the string matches |*field-content| in RFC 7230.
+  static bool IsValidHeaderValueRFC7230(const std::string& value);
+  static bool IsValidHeaderValueRFC7230(std::string::const_iterator value_begin,
+                                        std::string::const_iterator value_end);

[davidben, 2015/09/29 16:32:59]

This should use a base::StringPiece. Then you don't need to convert or have both
versions.

[hiroshige, 2015/09/30 08:19:23]

Done.

+
   // Strips all header lines from |headers| whose name matches
   // |headers_to_remove|. |headers_to_remove| is a list of null-terminated
   // lower-case header names, with array length |headers_to_remove_len|.
   // Returns the stripped header lines list, separated by "\r\n".
   static std::string StripHeaders(const std::string& headers,
                                   const char* const headers_to_remove[],
                                   size_t headers_to_remove_len);
 
   // Multiple occurances of some headers cannot be coalesced into a comma-
   // separated list since their values are (or contain) unquoted HTTP-date
@@ skipping 290 matching lines @@
 
     // True if values are required for each name/value pair; false if a
     // name is permitted to appear without a corresponding value.
     bool values_optional_;
   };
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_HTTP_UTIL_H_