commit 9faeded93d44c81c50c3d4ebfa3009c60f08a4c4
author willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Thu Apr 29 20:03:05 2010
committer willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Thu Apr 29 20:03:05 2010
tree 8cd8f6756d8af0a9c538fc1599744affa53b4fe9
parent 9f8980986fed2a3dbd518728153fce41ed4b18a7

Fix Alternate-Protocol to work with proxies.
Does not yet handle rewriting the URL for the PAC resolver.
BUG=42666

Review URL: http://codereview.chromium.org/1699023

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45976 0039d316-1c4b-4281-b951-d872f2087c98

net/http/http_network_transaction_unittest.cc

diff --git a/net/http/http_network_transaction_unittest.cc b/net/http/http_network_transaction_unittest.cc
index 7d6db6f1..1e012942 100644
--- a/net/http/http_network_transaction_unittest.cc
+++ b/net/http/http_network_transaction_unittest.cc
@@ -3805,12 +3805,11 @@
       "www.google.com:443",
     },
 
-// TODO(willchan): Uncomment these tests when they work.
-//    {
-//      "http_proxy",
-//      "http://host.with.alternate/direct",
-//      "proxy/http_proxy:80/host.with.alternate:443",
-//    },
+    {
+      "http_proxy",
+      "http://host.with.alternate/direct",
+      "host.with.alternate:443",
+    },
   };
 
   HttpNetworkTransaction::SetUseAlternateProtocols(true);
@@ -3860,12 +3859,11 @@
       "socks5/www.google.com:443",
     },
 
-// TODO(willchan): Uncomment these tests when they work.
-//    {
-//      "socks4://socks_proxy:1080",
-//      "http://host.with.alternate/direct",
-//      "proxy/socks4://socks_proxy:1080/host.with.alternate:443",
-//    },
+    {
+      "socks4://socks_proxy:1080",
+      "http://host.with.alternate/direct",
+      "socks4/host.with.alternate:443",
+    },
   };
 
   HttpNetworkTransaction::SetUseAlternateProtocols(true);