Bypass DRP if a redirect cycle is detected

components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.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_protocol.h"
 
 #include <vector>
 
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 87 matching lines @@
       request->proxy_server().is_direct() ||
       request->proxy_server().host_port_pair().IsEmpty()) {
     ReportResponseProxyServerStatusHistogram(
         RESPONSE_PROXY_SERVER_STATUS_EMPTY);
     return false;
   }
 
   DataReductionProxyTypeInfo data_reduction_proxy_type_info;
   if (!config_->WasDataReductionProxyUsed(request,
                                           &data_reduction_proxy_type_info)) {
-    if (!HasDataReductionProxyViaHeader(response_headers, nullptr)) {
+    if (!HasDataReductionProxyViaHeader(*response_headers, nullptr)) {
       ReportResponseProxyServerStatusHistogram(
           RESPONSE_PROXY_SERVER_STATUS_NON_DRP_NO_VIA);
       return false;
     }
     ReportResponseProxyServerStatusHistogram(
         RESPONSE_PROXY_SERVER_STATUS_NON_DRP_WITH_VIA);
 
     // If the |proxy_server| doesn't match any of the currently configured Data
     // Reduction Proxies, but it still has the Data Reduction Proxy via header,
     // then apply the bypass logic regardless.
@@ skipping 12 matching lines @@
   } else {
     ReportResponseProxyServerStatusHistogram(RESPONSE_PROXY_SERVER_STATUS_DRP);
   }
 
   if (data_reduction_proxy_type_info.proxy_servers.empty())
     return false;
 
   // At this point, the response is expected to have the data reduction proxy
   // via header, so detect and report cases where the via header is missing.
   DataReductionProxyBypassStats::DetectAndRecordMissingViaHeaderResponseCode(
-      data_reduction_proxy_type_info.proxy_index == 0, response_headers);
+      data_reduction_proxy_type_info.proxy_index == 0, *response_headers);
 
   // GetDataReductionProxyBypassType will only log a net_log event if a bypass
   // command was sent via the data reduction proxy headers
   DataReductionProxyBypassType bypass_type = GetDataReductionProxyBypassType(
-      response_headers, data_reduction_proxy_info);
+      request->url_chain(), *response_headers, data_reduction_proxy_info);
 
   if (proxy_bypass_type)
     *proxy_bypass_type = bypass_type;
   if (bypass_type == BYPASS_EVENT_TYPE_MAX)
     return false;
 
   DCHECK(request->context());
   DCHECK(request->context()->proxy_service());
   net::ProxyServer proxy_server =
       data_reduction_proxy_type_info.proxy_servers.front();
@@ skipping 16 matching lines @@
                           net::LOAD_DISABLE_CACHE |
                           net::LOAD_BYPASS_PROXY);
   }
 
   // Retry if block-once was specified or if method is idempotent.
   return bypass_type == BYPASS_EVENT_TYPE_CURRENT ||
          util::IsMethodIdempotent(request->method());
 }
 
 }  // namespace data_reduction_proxy