Bypass DRP if a redirect cycle is detected

components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.cc

 // Copyright 2014 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 "components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.h"
 
 #include "base/callback.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/metrics/sparse_histogram.h"
 #include "components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h"
@@ skipping 82 matching lines @@
     } else {
       UMA_HISTOGRAM_ENUMERATION("DataReductionProxy.BypassTypeFallback",
                                 bypass_type, BYPASS_EVENT_TYPE_MAX);
     }
   }
 }
 
 // static
 void DataReductionProxyBypassStats::DetectAndRecordMissingViaHeaderResponseCode(
     bool is_primary,
-    const net::HttpResponseHeaders* headers) {
-  if (HasDataReductionProxyViaHeader(headers, NULL)) {
+    const net::HttpResponseHeaders& headers) {
+  if (HasDataReductionProxyViaHeader(headers, nullptr)) {
     // The data reduction proxy via header is present, so don't record anything.
     return;
   }
 
   if (is_primary) {
     UMA_HISTOGRAM_SPARSE_SLOWLY(
         "DataReductionProxy.MissingViaHeader.ResponseCode.Primary",
-        headers->response_code());
+        headers.response_code());
   } else {
     UMA_HISTOGRAM_SPARSE_SLOWLY(
         "DataReductionProxy.MissingViaHeader.ResponseCode.Fallback",
-        headers->response_code());
+        headers.response_code());
   }
 }
 
 DataReductionProxyBypassStats::DataReductionProxyBypassStats(
     DataReductionProxyConfig* config,
     UnreachableCallback unreachable_callback)
     : data_reduction_proxy_config_(config),
       unreachable_callback_(unreachable_callback),
       last_bypass_type_(BYPASS_EVENT_TYPE_MAX),
       triggering_request_(true),
@@ skipping 251 matching lines @@
 
 void DataReductionProxyBypassStats::RecordMissingViaHeaderBytes(
     const net::URLRequest& request) {
   DCHECK(thread_checker_.CalledOnValidThread());
   // Responses that were served from cache should have been filtered out
   // already.
   DCHECK(!request.was_cached());
 
   if (!data_reduction_proxy_config_->WasDataReductionProxyUsed(&request,
                                                                NULL) ||
-      HasDataReductionProxyViaHeader(request.response_headers(), NULL)) {
+      HasDataReductionProxyViaHeader(*request.response_headers(), NULL)) {
     // Only track requests that used the data reduction proxy and had responses
     // that were missing the data reduction proxy via header.
     return;
   }
 
   if (request.GetResponseCode() >= net::HTTP_BAD_REQUEST &&
       request.GetResponseCode() < net::HTTP_INTERNAL_SERVER_ERROR) {
     // Track 4xx responses that are missing via headers separately.
     UMA_HISTOGRAM_COUNTS("DataReductionProxy.MissingViaHeader.Bytes.4xx",
                          request.received_response_content_length());
@@ skipping 131 matching lines @@
           UMA_HISTOGRAM_COUNTS(
               "DataReductionProxy.BypassedBytes.Status502HttpBadGateway",
               content_length);
           break;
         case BYPASS_EVENT_TYPE_STATUS_503_HTTP_SERVICE_UNAVAILABLE:
           UMA_HISTOGRAM_COUNTS(
               "DataReductionProxy.BypassedBytes."
               "Status503HttpServiceUnavailable",
               content_length);
           break;
+        case BYPASS_EVENT_TYPE_URL_REDIRECT_CYCLE:
+          UMA_HISTOGRAM_COUNTS(
+              "DataReductionProxy.BypassedBytes.URLRedirectCycle",
+              content_length);
+          break;
         default:
           break;
       }
       break;
   }
 }
 
 }  // namespace data_reduction_proxy