Pass in bad proxy to NotifyProxyFallback and not the successful one.

net/proxy/proxy_service.cc

Index: net/proxy/proxy_service.cc
diff --git a/net/proxy/proxy_service.cc b/net/proxy/proxy_service.cc
index 4c59024746da2662e643bf8f4568d0c9a124d645..9b0b731c637f6b2734483a3801bd3fdc51a3ef91 100644
--- a/net/proxy/proxy_service.cc
+++ b/net/proxy/proxy_service.cc
@@ -1249,8 +1249,10 @@ void ProxyService::ReportSuccess(const ProxyInfo& result,
     if (existing == proxy_retry_info_.end()) {
       proxy_retry_info_[iter->first] = iter->second;
       if (network_delegate) {
+        const ProxyServer& bad_proxy =
+            ProxyServer::FromURI(iter->first, ProxyServer::SCHEME_HTTP);
         const ProxyRetryInfo& proxy_retry_info = iter->second;

[bengr, 2014/09/09 20:43:30]

Is it correct to always use SCHEME_HTTP? 

[Not at Google. Contact bengr, 2014/09/09 20:51:29]

SCHEME_HTTP is the default used by the net stack as far as I can see. For
example, see
https://code.google.com/p/chromium/codesearch#chromium/src/net/proxy/proxy_li....

-        network_delegate->NotifyProxyFallback(result.proxy_server(),
+        network_delegate->NotifyProxyFallback(bad_proxy,
                                               proxy_retry_info.net_error);
       }
     }