Bypassed Bytes UMAs

components/data_reduction_proxy/browser/data_reduction_proxy_params.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/browser/data_reduction_proxy_params.h"
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "components/data_reduction_proxy/common/data_reduction_proxy_switches.h"
 #include "net/proxy/proxy_info.h"
@@ skipping 311 matching lines @@
     return std::string();
   if (command_line.HasSwitch(switches::kEnableDataReductionProxyDev) ||
       (FieldTrialList::FindFullName("DataCompressionProxyDevRollout") ==
          kEnabled)) {
     return DATA_REDUCTION_DEV_HOST;
   }
 #endif
   return std::string();
 }
 
+bool DataReductionProxyParams::WereDataReductionProxiesBypassed(
+    const net::URLRequest* request) const {
+  DataReductionProxyParams::DataReductionProxyList proxies =
+      GetAllowedProxies();
+   if (proxies.size() == 0)
+     return false;
+
+   if (request == NULL || request->context() == NULL ||

[bengr, 2014/07/16 01:40:09]

Are any of these NULL in practice? I'd prefer:
DCHECK(request);
DCHECK(request->context());
DCHECK(request->context()->proxy_service());

+       request->context()->proxy_service() == NULL) {
+     return false;
+   }
+
+   const net::ProxyRetryInfoMap& retry_map =
+       request->context()->proxy_service()->proxy_retry_info();
+   if (retry_map.size() == 0)
+     return false;
+
+   // The request is bypassed if all configured proxies are in the retry map.
+   for (size_t i = 0; i < proxies.size(); ++i) {

[bengr, 2014/07/16 01:40:10]

This isn't right if alt proxies are involved. If they are, the data reduction
proxy is bypassed if:

1) both origin() and fallback_origin() appear in the retry map, or
2) both alt_origin() and alt_fallback_origin() appear in the map and the request
scheme is http, or
3) ssl_origin() is in the map and the scheme is https.

[megjablon, 2014/07/16 23:07:14]

This is based on: 

https://code.google.com/p/chromium/codesearch#chromium/src/components/data_re...

Does that need to be fixed too?

[bengr, 2014/07/16 23:28:14]

Yes

+     std::string proxy = net::HostPortPair::FromURL(proxies[i]).ToString();
+     // The retry list has the scheme prefix for https but not for http.
+     if (proxies[i].SchemeIs("https"))
+       proxy = std::string("https://") + proxy;
+
+     net::ProxyRetryInfoMap::const_iterator found = retry_map.find(proxy);
+     if (found == retry_map.end())
+       return false;
+
+   }
+   return true;
+}
+
 std::string DataReductionProxyParams::GetDefaultOrigin() const {
 #if defined(SPDY_PROXY_AUTH_ORIGIN)
   return SPDY_PROXY_AUTH_ORIGIN;
 #endif
   return std::string();
 }
 
 std::string DataReductionProxyParams::GetDefaultFallbackOrigin() const {
 #if defined(DATA_REDUCTION_FALLBACK_HOST)
   return DATA_REDUCTION_FALLBACK_HOST;
@@ skipping 30 matching lines @@
 }
 
 std::string DataReductionProxyParams::GetDefaultWarmupURL() const {
 #if defined(DATA_REDUCTION_PROXY_WARMUP_URL)
   return DATA_REDUCTION_PROXY_WARMUP_URL;
 #endif
   return std::string();
 }
 
 }  // namespace data_reduction_proxy