Add data reduction functionality to all platforms.

net/proxy/proxy_list.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/proxy/proxy_list.h"
 
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/rand_util.h"
 #include "base/strings/string_tokenizer.h"
@@ skipping 149 matching lines @@
   //
   // And also, before failing the transaction wholesale, we could go back and
   // retry the "bad proxies" which we never tried to begin with.
   // (RemoveBadProxies would annotate them as 'expected bad' rather then delete
   // them from the list, so we would know what they were).
 
   if (proxies_.empty()) {
     NOTREACHED();
     return false;
   }
-  UpdateRetryInfoOnFallback(proxy_retry_info, base::TimeDelta(), true,
+  UpdateRetryInfoOnFallback(proxy_retry_info, TimeDelta::FromMinutes(5), true,
                             ProxyServer(), net_log);
 
   // Remove this proxy from our list.
   proxies_.erase(proxies_.begin());
   return !proxies_.empty();
 }
 
 void ProxyList::AddProxyToRetryList(ProxyRetryInfoMap* proxy_retry_info,
                                     base::TimeDelta retry_delay,
                                     bool try_while_bad,
@@ skipping 16 matching lines @@
   net_log.AddEvent(NetLog::TYPE_PROXY_LIST_FALLBACK,
                    NetLog::StringCallback("bad_proxy", &proxy_key));
 }
 
 void ProxyList::UpdateRetryInfoOnFallback(
     ProxyRetryInfoMap* proxy_retry_info,
     base::TimeDelta retry_delay,
     bool reconsider,
     const ProxyServer& another_proxy_to_bypass,
     const BoundNetLog& net_log) const {
-  // Time to wait before retrying a bad proxy server.
-  if (retry_delay == base::TimeDelta()) {
-#if defined(SPDY_PROXY_AUTH_ORIGIN)
-    // Randomize the timeout over a range from one to five minutes.
-    retry_delay =
-        TimeDelta::FromMilliseconds(
-            base::RandInt(1 * 60 * 1000, 5 * 60 * 1000));
-#else
-    retry_delay = TimeDelta::FromMinutes(5);
-#endif
-  }
-
+  DCHECK(retry_delay != base::TimeDelta());

[bengr, 2014/07/15 16:38:17]

I think this is wrong too. 

I think there are three cases and you'll have to update the code accordingly:

(1) If the data reduction proxy sent a response that results in bypass,
retry_delay will be non-zero, and this function should use that delay.

(2) If a network error results in a bypass and the proxy being bypassed is the
data reduction proxy. retry_delay should be a time between one and five minutes.


(3) If a network error results in a bypass and the proxy being bypassed is not
the data reduction proxy. retry_delay should be five minutes. 

For (2) and (3) the values of retry_delay should be passed into this method, I
suppose. For (1), the values must be passed in.

[Not at Google. Contact bengr, 2014/07/16 22:41:34]

Fixed now.

   if (proxies_.empty()) {
     NOTREACHED();
     return;
   }
 
   if (!proxies_[0].is_direct()) {
     AddProxyToRetryList(proxy_retry_info, retry_delay, reconsider, proxies_[0],
                         net_log);
 
     // If an additional proxy to bypass is specified, add it to the retry map
     // as well.
     if (another_proxy_to_bypass.is_valid()) {
       AddProxyToRetryList(proxy_retry_info, retry_delay, reconsider,
                           another_proxy_to_bypass, net_log);
     }
   }
 }
 
 }  // namespace net