Check in patch for pierre.lafayette, http://codereview.chromium.org/178059/sh...

net/base/net_util_unittest.cc

Index: net/base/net_util_unittest.cc
===================================================================
--- net/base/net_util_unittest.cc	(revision 25619)
+++ net/base/net_util_unittest.cc	(working copy)
@@ -1313,3 +1313,19 @@
     EXPECT_EQ(expected_url, net::SimplifyUrlForRequest(input_url));
   }
 }
+
+TEST(NetUtilTest, SetExplicitlyAllowedPortsTest) {
+  std::wstring invalid[] = { L"1,2,a", L"'1','2'", L"1, 2, 3", L"1 0,11,12" };
+  std::wstring valid[] = { L"", L"1", L"1,2", L"1,2,3", L"10,11,12,13" };
+
+  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(invalid); ++i) {
+    net::SetExplicitlyAllowedPorts(invalid[i]);
+    EXPECT_EQ(0, static_cast<int>(net::explicitly_allowed_ports.size()));
+  }
+
+  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(valid); ++i) {
+    net::SetExplicitlyAllowedPorts(valid[i]);
+    EXPECT_EQ(i, net::explicitly_allowed_ports.size());
+  }
+}
+