Allow for an existing bad proxy to be given a new retry delay.

net/proxy/proxy_list_unittest.cc

 // Copyright (c) 2006-2008 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 <vector>
 
 #include "net/base/net_errors.h"
 #include "net/log/net_log.h"
@@ skipping 227 matching lines @@
                                                     ProxyServer::SCHEME_HTTP);
     std::vector<ProxyServer> bad_proxies;
     bad_proxies.push_back(proxy_server);
     list.SetFromPacString("DIRECT;PROXY foopy2:80;PROXY foopy3:80");
     list.UpdateRetryInfoOnFallback(&retry_info_map,
                                    base::TimeDelta::FromSeconds(60), true,
                                    bad_proxies, OK, net_log);
     EXPECT_TRUE(retry_info_map.end() == retry_info_map.find("foopy2:80"));
     EXPECT_TRUE(retry_info_map.end() == retry_info_map.find("foopy3:80"));
   }
+  // If the bad proxy is already on the retry list, and the old retry info would
+  // cause the proxy to be retried later than the newly specified retry info,
+  // then the old retry info should be kept.
+  {
+    ProxyList list;
+    ProxyRetryInfoMap retry_info_map;
+    BoundNetLog net_log;
+    list.SetFromPacString("PROXY foopy1:80;PROXY foopy2:80;PROXY foopy3:80");
+
+    // First, mark the proxy as bad for 60 seconds.
+    list.UpdateRetryInfoOnFallback(
+        &retry_info_map, base::TimeDelta::FromSeconds(60), true,
+        std::vector<ProxyServer>(), ERR_PROXY_CONNECTION_FAILED, net_log);
+    // Next, mark the same proxy as bad for 1 second. This call should have no
+    // effect, since this would cause the bad proxy to be retried sooner than
+    // the existing retry info.
+    list.UpdateRetryInfoOnFallback(&retry_info_map,
+                                   base::TimeDelta::FromSeconds(1), false,
+                                   std::vector<ProxyServer>(), OK, net_log);
+    EXPECT_TRUE(retry_info_map.end() != retry_info_map.find("foopy1:80"));
+    EXPECT_EQ(ERR_PROXY_CONNECTION_FAILED,
+              retry_info_map["foopy1:80"].net_error);
+    EXPECT_TRUE(retry_info_map["foopy1:80"].try_while_bad);
+    EXPECT_EQ(base::TimeDelta::FromSeconds(60),
+              retry_info_map["foopy1:80"].current_delay);
+    EXPECT_GT(retry_info_map["foopy1:80"].bad_until,
+              base::TimeTicks::Now() + base::TimeDelta::FromSeconds(30));
+    EXPECT_TRUE(retry_info_map.end() == retry_info_map.find("foopy2:80"));
+    EXPECT_TRUE(retry_info_map.end() == retry_info_map.find("foopy3:80"));
+  }
+  // If the bad proxy is already on the retry list, and the newly specified
+  // retry info would cause the proxy to be retried later than the old retry
+  // info, then the old retry info should be replaced with the new retry info.
+  {
+    ProxyList list;
+    ProxyRetryInfoMap retry_info_map;
+    BoundNetLog net_log;
+    list.SetFromPacString("PROXY foopy1:80;PROXY foopy2:80;PROXY foopy3:80");
+
+    // First, mark the proxy as bad for 1 second.
+    list.UpdateRetryInfoOnFallback(&retry_info_map,
+                                   base::TimeDelta::FromSeconds(1), false,
+                                   std::vector<ProxyServer>(), OK, net_log);
+    // Next, mark the same proxy as bad for 60 seconds. This call should replace
+    // the existing retry info with the new 60 second retry info.
+    list.UpdateRetryInfoOnFallback(
+        &retry_info_map, base::TimeDelta::FromSeconds(60), true,
+        std::vector<ProxyServer>(), ERR_PROXY_CONNECTION_FAILED, net_log);
+
+    EXPECT_TRUE(retry_info_map.end() != retry_info_map.find("foopy1:80"));
+    EXPECT_EQ(ERR_PROXY_CONNECTION_FAILED,
+              retry_info_map["foopy1:80"].net_error);
+    EXPECT_TRUE(retry_info_map["foopy1:80"].try_while_bad);
+    EXPECT_EQ(base::TimeDelta::FromSeconds(60),
+              retry_info_map["foopy1:80"].current_delay);
+    EXPECT_GT(retry_info_map["foopy1:80"].bad_until,
+              base::TimeTicks::Now() + base::TimeDelta::FromSeconds(30));
+    EXPECT_TRUE(retry_info_map.end() == retry_info_map.find("foopy2:80"));
+    EXPECT_TRUE(retry_info_map.end() == retry_info_map.find("foopy3:80"));
+  }
 }
 
 }  // namesapce
 
 }  // namespace net