Deprecate Net.{HttpRequest,Response}.ContainsInvalidHeaderValuesInRFC7230 UMAs

net/url_request/url_request_http_job.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.
 
 #include "net/url_request/url_request_http_job.h"
 
 #include "base/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
@@ skipping 592 matching lines @@
 
 void URLRequestHttpJob::StartTransactionInternal() {
   // This should only be called while the request's status is IO_PENDING.
   DCHECK_EQ(URLRequestStatus::IO_PENDING, request_->status().status());
 
   // NOTE: This method assumes that request_info_ is already setup properly.
 
   // If we already have a transaction, then we should restart the transaction
   // with auth provided by auth_credentials_.
 
-  bool invalid_header_values_in_rfc7230 = false;
-  for (HttpRequestHeaders::Iterator it(request_info_.extra_headers);
-       it.GetNext();) {
-    if (!HttpUtil::IsValidHeaderValueRFC7230(it.value())) {
-      invalid_header_values_in_rfc7230 = true;
-      break;
-    }
-  }
-  UMA_HISTOGRAM_BOOLEAN("Net.HttpRequest.ContainsInvalidHeaderValuesInRFC7230",
-                        invalid_header_values_in_rfc7230);
-
   int rv;
 
   // Notify NetworkQualityEstimator.
   NetworkQualityEstimator* network_quality_estimator =
       request()->context()->network_quality_estimator();
   if (network_quality_estimator)
     network_quality_estimator->NotifyStartTransaction(*request_);
 
   if (network_delegate()) {
     network_delegate()->NotifyStartTransaction(request_,
@@ skipping 392 matching lines @@
   receive_headers_end_ = base::TimeTicks::Now();
 
   const URLRequestContext* context = request_->context();
 
   if (result == OK) {
     if (transaction_ && transaction_->GetResponseInfo()) {
       SetProxyServer(transaction_->GetResponseInfo()->proxy_server);
     }
     scoped_refptr<HttpResponseHeaders> headers = GetResponseHeaders();
 
-    if (headers) {
-      size_t iter = 0;
-      std::string name;
-      std::string value;
-      bool invalid_header_values_in_rfc7230 = false;
-      while (headers->EnumerateHeaderLines(&iter, &name, &value)) {
-        if (!HttpUtil::IsValidHeaderValueRFC7230(value)) {
-          invalid_header_values_in_rfc7230 = true;
-          break;
-        }
-      }
-      UMA_HISTOGRAM_BOOLEAN(
-          "Net.HttpResponse.ContainsInvalidHeaderValuesInRFC7230",
-          invalid_header_values_in_rfc7230);
-    }
-
     if (network_delegate()) {
       // Note that |this| may not be deleted until
       // |on_headers_received_callback_| or
       // |NetworkDelegate::URLRequestDestroyed()| has been called.
       OnCallToDelegate();
       allowed_unsafe_redirect_url_ = GURL();
       int error = network_delegate()->NotifyHeadersReceived(
           request_,
           on_headers_received_callback_,
           headers.get(),
@@ skipping 616 matching lines @@
   // Notify NetworkQualityEstimator.
   if (request()) {
     NetworkQualityEstimator* network_quality_estimator =
         request()->context()->network_quality_estimator();
     if (network_quality_estimator)
       network_quality_estimator->NotifyURLRequestDestroyed(*request());
   }
 }
 
 }  // namespace net