This fixes http://code.google.com/p/chromium/issues/detail?id=541, which is an

issue with the proxy exception list not being honored if individual rules had
uppercase characters in them.

The code to match the url with the exceptions was converting both to lowercase.
However there was an error in the way the StringToLowerASCII function was used.

Bug=541

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

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

1 files changed, 16 insertions, 0 deletions

diff --git a/net/proxy/proxy_service_unittest.cc b/net/proxy/proxy_service_unittest.cc
index 5731955..d33ac6d 100644
--- a/net/proxy/proxy_service_unittest.cc
+++ b/net/proxy/proxy_service_unittest.cc
@@ -341,6 +341,22 @@ TEST(ProxyServiceTest, ProxyBypassList) {
   rv = service4.ResolveProxy(test_url4, &info4, NULL, NULL);
   EXPECT_EQ(rv, net::OK);
   EXPECT_FALSE(info4.is_direct());
+
+  resolver.config->proxy_bypass = "<local>;*.MSN.COM;";
+  net::ProxyService service5(&resolver);
+  GURL test_url5("http://www.msnbc.msn.com");
+  net::ProxyInfo info5;
+  rv = service5.ResolveProxy(test_url5, &info5, NULL, NULL);
+  EXPECT_EQ(rv, net::OK);
+  EXPECT_TRUE(info5.is_direct());
+
+  resolver.config->proxy_bypass = "<local>;*.msn.com;";
+  net::ProxyService service6(&resolver);
+  GURL test_url6("HTTP://WWW.MSNBC.MSN.COM");
+  net::ProxyInfo info6;
+  rv = service6.ResolveProxy(test_url6, &info6, NULL, NULL);
+  EXPECT_EQ(rv, net::OK);
+  EXPECT_TRUE(info6.is_direct());
 }
 
 TEST(ProxyServiceTest, PerProtocolProxyTests) {