Add data reduction functionality to all platforms.

net/proxy/proxy_list.cc

Index: net/proxy/proxy_list.cc
diff --git a/net/proxy/proxy_list.cc b/net/proxy/proxy_list.cc
index 4bbb6bc6a96f2d021c66b76f333b640a405ef5d0..127dc275608fd03dc541f1c413d7f5f58fb76588 100644
--- a/net/proxy/proxy_list.cc
+++ b/net/proxy/proxy_list.cc
@@ -167,7 +167,7 @@ bool ProxyList::Fallback(ProxyRetryInfoMap* proxy_retry_info,
     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.
@@ -204,18 +204,7 @@ void ProxyList::UpdateRetryInfoOnFallback(
     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;