net: remove AppendHeaderIfMissing() from HttpUtil

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 294 matching lines @@
     return false;
   }
 
   if (interval < base::TimeDelta::FromSeconds(0))
     return false;
 
   *retry_after = interval;
   return true;
 }
 
-// static
-bool HttpUtil::HasHeader(const std::string& headers, const char* name) {
-  size_t name_len = strlen(name);
-  std::string::const_iterator it =
-      std::search(headers.begin(),
-                  headers.end(),
-                  name,
-                  name + name_len,
-                  base::CaseInsensitiveCompareASCII<char>());
-  if (it == headers.end())
-    return false;
-
-  // ensure match is prefixed by newline
-  if (it != headers.begin() && it[-1] != '\n')
-    return false;
-
-  // ensure match is suffixed by colon
-  if (it + name_len >= headers.end() || it[name_len] != ':')
-    return false;
-
-  return true;
-}
-
 namespace {
+
 // A header string containing any of the following fields will cause
 // an error. The list comes from the XMLHttpRequest standard.
 // http://www.w3.org/TR/XMLHttpRequest/#the-setrequestheader-method
 const char* const kForbiddenHeaderFields[] = {
   "accept-charset",
   "accept-encoding",
   "access-control-request-headers",
   "access-control-request-method",
   "connection",
   "content-length",
   "cookie",
   "cookie2",
   "content-transfer-encoding",
   "date",
   "expect",
   "host",
   "keep-alive",
   "origin",
   "referer",
   "te",
   "trailer",
   "transfer-encoding",
   "upgrade",
   "user-agent",
   "via",
 };
-}  // anonymous namespace
+
+}  // namespace
 
 // static
 bool HttpUtil::IsSafeHeader(const std::string& name) {
   std::string lower_name(base::ToLowerASCII(name));
   if (base::StartsWith(lower_name, "proxy-", base::CompareCase::SENSITIVE) ||
       base::StartsWith(lower_name, "sec-", base::CompareCase::SENSITIVE))
     return false;
 
   for (const char* field : kForbiddenHeaderFields) {
     if (lower_name == field)
@@ skipping 398 matching lines @@
       base::StringAppendF(&lang_list_with_q, ",%s;q=0.%d", language.c_str(),
                           qvalue10);
     }
     // It does not make sense to have 'q=0'.
     if (qvalue10 > kQvalueDecrement10)
       qvalue10 -= kQvalueDecrement10;
   }
   return lang_list_with_q;
 }
 
-void HttpUtil::AppendHeaderIfMissing(const char* header_name,
-                                     const std::string& header_value,
-                                     std::string* headers) {
-  if (header_value.empty())
-    return;
-  if (HttpUtil::HasHeader(*headers, header_name))
-    return;
-  *headers += std::string(header_name) + ": " + header_value + "\r\n";
-}
-
 bool HttpUtil::HasStrongValidators(HttpVersion version,
                                    const std::string& etag_header,
                                    const std::string& last_modified_header,
                                    const std::string& date_header) {
   if (!HasValidators(version, etag_header, last_modified_header))
     return false;
 
   if (version < HttpVersion(1, 1))
     return false;
 
@@ skipping 269 matching lines @@
   return true;
 }
 
 bool HttpUtil::NameValuePairsIterator::IsQuote(char c) const {
   if (strict_quotes_)
     return c == '"';
   return HttpUtil::IsQuote(c);
 }
 
 }  // namespace net