Split network_interfaces.h off of net_utils.h.

net/base/network_interfaces_unittest.cc

Index: net/base/network_interfaces_unittest.cc
diff --git a/net/base/net_util_unittest.cc b/net/base/network_interfaces_unittest.cc
similarity index 59%
copy from net/base/net_util_unittest.cc
copy to net/base/network_interfaces_unittest.cc
index 5e138d5937e1e4b3b1860e9a61fd0c6fa6c744b9..4cc2f545599147afe51f298d0fde3327c04f8cf2 100644
--- a/net/base/net_util_unittest.cc
+++ b/net/base/network_interfaces_unittest.cc
@@ -2,12 +2,13 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "net/base/net_util.h"
+#include "net/base/network_interfaces.h"
 
-#include <string.h>
+#include <string>
 
 #include <ostream>
 
+// TODO(eroman): Remove unneeeded headers.
 #include "base/files/file_path.h"
 #include "base/format_macros.h"
 #include "base/scoped_native_library.h"
@@ -43,28 +44,20 @@
 #endif  // !OS_MACOSX && !OS_NACL && !OS_WIN
 
 #if defined(OS_WIN)
-#include "net/base/net_util_win.h"
+#include "net/base/network_interfaces_win.h"
 #else  // OS_WIN
-#include "net/base/net_util_posix.h"
+#include "net/base/network_interfaces_posix.h"
 #if defined(OS_MACOSX)
-#include "net/base/net_util_mac.h"
+#include "net/base/network_interfaces_mac.h"
 #else  // OS_MACOSX
-#include "net/base/net_util_linux.h"
+#include "net/base/network_interfaces_linux.h"
 #endif  // OS_MACOSX
 #endif  // OS_WIN
 
-using base::ASCIIToUTF16;
-using base::WideToUTF16;
-
 namespace net {
 
 namespace {
 
-struct HeaderCase {
-  const char* const header_name;
-  const char* const expected;
-};
-
 #if defined(OS_LINUX) || defined(OS_ANDROID) || defined(OS_CHROMEOS)
 const char kWiFiSSID[] = "TestWiFi";
 const char kInterfaceWithDifferentSSID[] = "wlan999";
@@ -74,28 +67,6 @@ std::string TestGetInterfaceSSID(const std::string& ifname) {
 }
 #endif
 
-// Fills in sockaddr for the given 32-bit address (IPv4.)
-// |bytes| should be an array of length 4.
-void MakeIPv4Address(const uint8* bytes, int port, SockaddrStorage* storage) {
-  memset(&storage->addr_storage, 0, sizeof(storage->addr_storage));
-  storage->addr_len = sizeof(struct sockaddr_in);
-  struct sockaddr_in* addr4 = reinterpret_cast<sockaddr_in*>(storage->addr);
-  addr4->sin_port = base::HostToNet16(port);
-  addr4->sin_family = AF_INET;
-  memcpy(&addr4->sin_addr, bytes, 4);
-}
-
-// Fills in sockaddr for the given 128-bit address (IPv6.)
-// |bytes| should be an array of length 16.
-void MakeIPv6Address(const uint8* bytes, int port, SockaddrStorage* storage) {
-  memset(&storage->addr_storage, 0, sizeof(storage->addr_storage));
-  storage->addr_len = sizeof(struct sockaddr_in6);
-  struct sockaddr_in6* addr6 = reinterpret_cast<sockaddr_in6*>(storage->addr);
-  addr6->sin6_port = base::HostToNet16(port);
-  addr6->sin6_family = AF_INET6;
-  memcpy(&addr6->sin6_addr, bytes, 16);
-}
-
 #if defined(OS_MACOSX)
 class IPAttributesGetterTest : public internal::IPAttributesGetterMac {
  public:
@@ -147,482 +118,6 @@ bool FillIfaddrs(ifaddrs* interfaces,
   return true;
 }
 #endif  // OS_MACOSX
-
-}  // anonymous namespace
-
-TEST(NetUtilTest, GetIdentityFromURL) {
-  struct {
-    const char* const input_url;
-    const char* const expected_username;
-    const char* const expected_password;
-  } tests[] = {
-    {
-      "http://username:password@google.com",
-      "username",
-      "password",
-    },
-    { // Test for http://crbug.com/19200
-      "http://username:p@ssword@google.com",
-      "username",
-      "p@ssword",
-    },
-    { // Special URL characters should be unescaped.
-      "http://username:p%3fa%26s%2fs%23@google.com",
-      "username",
-      "p?a&s/s#",
-    },
-    { // Username contains %20.
-      "http://use rname:password@google.com",
-      "use rname",
-      "password",
-    },
-    { // Keep %00 as is.
-      "http://use%00rname:password@google.com",
-      "use%00rname",
-      "password",
-    },
-    { // Use a '+' in the username.
-      "http://use+rname:password@google.com",
-      "use+rname",
-      "password",
-    },
-    { // Use a '&' in the password.
-      "http://username:p&ssword@google.com",
-      "username",
-      "p&ssword",
-    },
-  };
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    SCOPED_TRACE(base::StringPrintf("Test[%" PRIuS "]: %s", i,
-                                    tests[i].input_url));
-    GURL url(tests[i].input_url);
-
-    base::string16 username, password;
-    GetIdentityFromURL(url, &username, &password);
-
-    EXPECT_EQ(ASCIIToUTF16(tests[i].expected_username), username);
-    EXPECT_EQ(ASCIIToUTF16(tests[i].expected_password), password);
-  }
-}
-
-// Try extracting a username which was encoded with UTF8.
-TEST(NetUtilTest, GetIdentityFromURL_UTF8) {
-  GURL url(WideToUTF16(L"http://foo:\x4f60\x597d@blah.com"));
-
-  EXPECT_EQ("foo", url.username());
-  EXPECT_EQ("%E4%BD%A0%E5%A5%BD", url.password());
-
-  // Extract the unescaped identity.
-  base::string16 username, password;
-  GetIdentityFromURL(url, &username, &password);
-
-  // Verify that it was decoded as UTF8.
-  EXPECT_EQ(ASCIIToUTF16("foo"), username);
-  EXPECT_EQ(WideToUTF16(L"\x4f60\x597d"), password);
-}
-
-// Just a bunch of fake headers.
-const char google_headers[] =
-    "HTTP/1.1 200 OK\n"
-    "Content-TYPE: text/html; charset=utf-8\n"
-    "Content-disposition: attachment; filename=\"download.pdf\"\n"
-    "Content-Length: 378557\n"
-    "X-Google-Google1: 314159265\n"
-    "X-Google-Google2: aaaa2:7783,bbb21:9441\n"
-    "X-Google-Google4: home\n"
-    "Transfer-Encoding: chunked\n"
-    "Set-Cookie: HEHE_AT=6666x66beef666x6-66xx6666x66; Path=/mail\n"
-    "Set-Cookie: HEHE_HELP=owned:0;Path=/\n"
-    "Set-Cookie: S=gmail=Xxx-beefbeefbeef_beefb:gmail_yj=beefbeef000beefbee"
-       "fbee:gmproxy=bee-fbeefbe; Domain=.google.com; Path=/\n"
-    "X-Google-Google2: /one/two/three/four/five/six/seven-height/nine:9411\n"
-    "Server: GFE/1.3\n"
-    "Transfer-Encoding: chunked\n"
-    "Date: Mon, 13 Nov 2006 21:38:09 GMT\n"
-    "Expires: Tue, 14 Nov 2006 19:23:58 GMT\n"
-    "X-Malformed: bla; arg=test\"\n"
-    "X-Malformed2: bla; arg=\n"
-    "X-Test: bla; arg1=val1; arg2=val2";
-
-TEST(NetUtilTest, GetSpecificHeader) {
-  const HeaderCase tests[] = {
-    {"content-type", "text/html; charset=utf-8"},
-    {"CONTENT-LENGTH", "378557"},
-    {"Date", "Mon, 13 Nov 2006 21:38:09 GMT"},
-    {"Bad-Header", ""},
-    {"", ""},
-  };
-
-  // Test first with google_headers.
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    std::string result =
-        GetSpecificHeader(google_headers, tests[i].header_name);
-    EXPECT_EQ(result, tests[i].expected);
-  }
-
-  // Test again with empty headers.
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    std::string result = GetSpecificHeader(std::string(), tests[i].header_name);
-    EXPECT_EQ(result, std::string());
-  }
-}
-
-TEST(NetUtilTest, CompliantHost) {
-  struct CompliantHostCase {
-    const char* const host;
-    bool expected_output;
-  };
-
-  const CompliantHostCase compliant_host_cases[] = {
-      {"", false},
-      {"a", true},
-      {"-", false},
-      {"_", false},
-      {".", false},
-      {"9", true},
-      {"9a", true},
-      {"9_", true},
-      {"a.", true},
-      {"a.a", true},
-      {"9.a", true},
-      {"a.9", true},
-      {"_9a", false},
-      {"-9a", false},
-      {"a.a9", true},
-      {"_.9a", true},
-      {"a.-a9", false},
-      {"a+9a", false},
-      {"-a.a9", true},
-      {"a_.a9", true},
-      {"1-.a-b", true},
-      {"1_.a-b", true},
-      {"1-2.a_b", true},
-      {"a.b.c.d.e", true},
-      {"1.2.3.4.5", true},
-      {"1.2.3.4.5.", true},
-  };
-
-  for (size_t i = 0; i < arraysize(compliant_host_cases); ++i) {
-    EXPECT_EQ(compliant_host_cases[i].expected_output,
-              IsCanonicalizedHostCompliant(compliant_host_cases[i].host));
-  }
-}
-
-TEST(NetUtilTest, ParseHostAndPort) {
-  const struct {
-    const char* const input;
-    bool success;
-    const char* const expected_host;
-    int expected_port;
-  } tests[] = {
-    // Valid inputs:
-    {"foo:10", true, "foo", 10},
-    {"foo", true, "foo", -1},
-    {
-      "[1080:0:0:0:8:800:200C:4171]:11",
-      true,
-      "1080:0:0:0:8:800:200C:4171",
-      11
-    },
-    {
-      "[1080:0:0:0:8:800:200C:4171]",
-      true,
-      "1080:0:0:0:8:800:200C:4171",
-      -1
-    },
-
-    // Because no validation is done on the host, the following are accepted,
-    // even though they are invalid names.
-    {"]", true, "]", -1},
-    {"::1", true, ":", 1},
-    // Invalid inputs:
-    {"foo:bar", false, "", -1},
-    {"foo:", false, "", -1},
-    {":", false, "", -1},
-    {":80", false, "", -1},
-    {"", false, "", -1},
-    {"porttoolong:300000", false, "", -1},
-    {"usrname@host", false, "", -1},
-    {"usrname:password@host", false, "", -1},
-    {":password@host", false, "", -1},
-    {":password@host:80", false, "", -1},
-    {":password@host", false, "", -1},
-    {"@host", false, "", -1},
-    {"[", false, "", -1},
-    {"[]", false, "", -1},
-  };
-
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    std::string host;
-    int port;
-    bool ok = ParseHostAndPort(tests[i].input, &host, &port);
-
-    EXPECT_EQ(tests[i].success, ok);
-
-    if (tests[i].success) {
-      EXPECT_EQ(tests[i].expected_host, host);
-      EXPECT_EQ(tests[i].expected_port, port);
-    }
-  }
-}
-
-TEST(NetUtilTest, GetHostAndPort) {
-  const struct {
-    GURL url;
-    const char* const expected_host_and_port;
-  } tests[] = {
-    { GURL("http://www.foo.com/x"), "www.foo.com:80"},
-    { GURL("http://www.foo.com:21/x"), "www.foo.com:21"},
-
-    // For IPv6 literals should always include the brackets.
-    { GURL("http://[1::2]/x"), "[1::2]:80"},
-    { GURL("http://[::a]:33/x"), "[::a]:33"},
-  };
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    std::string host_and_port = GetHostAndPort(tests[i].url);
-    EXPECT_EQ(std::string(tests[i].expected_host_and_port), host_and_port);
-  }
-}
-
-TEST(NetUtilTest, GetHostAndOptionalPort) {
-  const struct {
-    GURL url;
-    const char* const expected_host_and_port;
-  } tests[] = {
-    { GURL("http://www.foo.com/x"), "www.foo.com"},
-    { GURL("http://www.foo.com:21/x"), "www.foo.com:21"},
-
-    // For IPv6 literals should always include the brackets.
-    { GURL("http://[1::2]/x"), "[1::2]"},
-    { GURL("http://[::a]:33/x"), "[::a]:33"},
-  };
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    std::string host_and_port = GetHostAndOptionalPort(tests[i].url);
-    EXPECT_EQ(std::string(tests[i].expected_host_and_port), host_and_port);
-  }
-}
-
-TEST(NetUtilTest, NetAddressToString_IPv4) {
-  const struct {
-    uint8 addr[4];
-    const char* const result;
-  } tests[] = {
-    {{0, 0, 0, 0}, "0.0.0.0"},
-    {{127, 0, 0, 1}, "127.0.0.1"},
-    {{192, 168, 0, 1}, "192.168.0.1"},
-  };
-
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    SockaddrStorage storage;
-    MakeIPv4Address(tests[i].addr, 80, &storage);
-    std::string result = NetAddressToString(storage.addr, storage.addr_len);
-    EXPECT_EQ(std::string(tests[i].result), result);
-  }
-}
-
-TEST(NetUtilTest, NetAddressToString_IPv6) {
-  const struct {
-    uint8 addr[16];
-    const char* const result;
-  } tests[] = {
-    {{0xFE, 0xDC, 0xBA, 0x98, 0x76, 0x54, 0x32, 0x10, 0xFE, 0xDC, 0xBA,
-      0x98, 0x76, 0x54, 0x32, 0x10},
-     "fedc:ba98:7654:3210:fedc:ba98:7654:3210"},
-  };
-
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    SockaddrStorage storage;
-    MakeIPv6Address(tests[i].addr, 80, &storage);
-    EXPECT_EQ(std::string(tests[i].result),
-        NetAddressToString(storage.addr, storage.addr_len));
-  }
-}
-
-TEST(NetUtilTest, NetAddressToStringWithPort_IPv4) {
-  uint8 addr[] = {127, 0, 0, 1};
-  SockaddrStorage storage;
-  MakeIPv4Address(addr, 166, &storage);
-  std::string result = NetAddressToStringWithPort(storage.addr,
-                                                  storage.addr_len);
-  EXPECT_EQ("127.0.0.1:166", result);
-}
-
-TEST(NetUtilTest, NetAddressToStringWithPort_IPv6) {
-  uint8 addr[] = {
-      0xFE, 0xDC, 0xBA, 0x98, 0x76, 0x54, 0x32, 0x10, 0xFE, 0xDC, 0xBA,
-      0x98, 0x76, 0x54, 0x32, 0x10
-  };
-  SockaddrStorage storage;
-  MakeIPv6Address(addr, 361, &storage);
-  std::string result = NetAddressToStringWithPort(storage.addr,
-                                                  storage.addr_len);
-
-  // May fail on systems that don't support IPv6.
-  if (!result.empty())
-    EXPECT_EQ("[fedc:ba98:7654:3210:fedc:ba98:7654:3210]:361", result);
-}
-
-TEST(NetUtilTest, GetHostName) {
-  // We can't check the result of GetHostName() directly, since the result
-  // will differ across machines. Our goal here is to simply exercise the
-  // code path, and check that things "look about right".
-  std::string hostname = GetHostName();
-  EXPECT_FALSE(hostname.empty());
-}
-
-TEST(NetUtilTest, SimplifyUrlForRequest) {
-  struct {
-    const char* const input_url;
-    const char* const expected_simplified_url;
-  } tests[] = {
-    {
-      // Reference section should be stripped.
-      "http://www.google.com:78/foobar?query=1#hash",
-      "http://www.google.com:78/foobar?query=1",
-    },
-    {
-      // Reference section can itself contain #.
-      "http://192.168.0.1?query=1#hash#10#11#13#14",
-      "http://192.168.0.1?query=1",
-    },
-    { // Strip username/password.
-      "http://user:pass@google.com",
-      "http://google.com/",
-    },
-    { // Strip both the reference and the username/password.
-      "http://user:pass@google.com:80/sup?yo#X#X",
-      "http://google.com/sup?yo",
-    },
-    { // Try an HTTPS URL -- strip both the reference and the username/password.
-      "https://user:pass@google.com:80/sup?yo#X#X",
-      "https://google.com:80/sup?yo",
-    },
-    { // Try an FTP URL -- strip both the reference and the username/password.
-      "ftp://user:pass@google.com:80/sup?yo#X#X",
-      "ftp://google.com:80/sup?yo",
-    },
-    { // Try a nonstandard URL
-      "foobar://user:pass@google.com:80/sup?yo#X#X",
-      "foobar://user:pass@google.com:80/sup?yo",
-    },
-  };
-  for (size_t i = 0; i < arraysize(tests); ++i) {
-    SCOPED_TRACE(base::StringPrintf("Test[%" PRIuS "]: %s", i,
-                                    tests[i].input_url));
-    GURL input_url(GURL(tests[i].input_url));
-    GURL expected_url(GURL(tests[i].expected_simplified_url));
-    EXPECT_EQ(expected_url, SimplifyUrlForRequest(input_url));
-  }
-}
-
-TEST(NetUtilTest, SetExplicitlyAllowedPortsTest) {
-  std::string invalid[] = { "1,2,a", "'1','2'", "1, 2, 3", "1 0,11,12" };
-  std::string valid[] = { "", "1", "1,2", "1,2,3", "10,11,12,13" };
-
-  for (size_t i = 0; i < arraysize(invalid); ++i) {
-    SetExplicitlyAllowedPorts(invalid[i]);
-    EXPECT_EQ(0, static_cast<int>(GetCountOfExplicitlyAllowedPorts()));
-  }
-
-  for (size_t i = 0; i < arraysize(valid); ++i) {
-    SetExplicitlyAllowedPorts(valid[i]);
-    EXPECT_EQ(i, GetCountOfExplicitlyAllowedPorts());
-  }
-}
-
-TEST(NetUtilTest, GetHostOrSpecFromURL) {
-  EXPECT_EQ("example.com",
-            GetHostOrSpecFromURL(GURL("http://example.com/test")));
-  EXPECT_EQ("example.com",
-            GetHostOrSpecFromURL(GURL("http://example.com./test")));
-  EXPECT_EQ("file:///tmp/test.html",
-            GetHostOrSpecFromURL(GURL("file:///tmp/test.html")));
-}
-
-TEST(NetUtilTest, GetAddressFamily) {
-  IPAddressNumber number;
-  EXPECT_TRUE(ParseIPLiteralToNumber("192.168.0.1", &number));
-  EXPECT_EQ(ADDRESS_FAMILY_IPV4, GetAddressFamily(number));
-  EXPECT_TRUE(ParseIPLiteralToNumber("1:abcd::3:4:ff", &number));
-  EXPECT_EQ(ADDRESS_FAMILY_IPV6, GetAddressFamily(number));
-}
-
-TEST(NetUtilTest, IsLocalhost) {
-  EXPECT_TRUE(IsLocalhost("localhost"));
-  EXPECT_TRUE(IsLocalhost("localhost.localdomain"));
-  EXPECT_TRUE(IsLocalhost("localhost6"));
-  EXPECT_TRUE(IsLocalhost("localhost6.localdomain6"));
-  EXPECT_TRUE(IsLocalhost("127.0.0.1"));
-  EXPECT_TRUE(IsLocalhost("127.0.1.0"));
-  EXPECT_TRUE(IsLocalhost("127.1.0.0"));
-  EXPECT_TRUE(IsLocalhost("127.0.0.255"));
-  EXPECT_TRUE(IsLocalhost("127.0.255.0"));
-  EXPECT_TRUE(IsLocalhost("127.255.0.0"));
-  EXPECT_TRUE(IsLocalhost("::1"));
-  EXPECT_TRUE(IsLocalhost("0:0:0:0:0:0:0:1"));
-  EXPECT_TRUE(IsLocalhost("foo.localhost"));
-
-  EXPECT_FALSE(IsLocalhost("localhostx"));
-  EXPECT_FALSE(IsLocalhost("foo.localdomain"));
-  EXPECT_FALSE(IsLocalhost("localhost6x"));
-  EXPECT_FALSE(IsLocalhost("localhost.localdomain6"));
-  EXPECT_FALSE(IsLocalhost("localhost6.localdomain"));
-  EXPECT_FALSE(IsLocalhost("127.0.0.1.1"));
-  EXPECT_FALSE(IsLocalhost(".127.0.0.255"));
-  EXPECT_FALSE(IsLocalhost("::2"));
-  EXPECT_FALSE(IsLocalhost("::1:1"));
-  EXPECT_FALSE(IsLocalhost("0:0:0:0:1:0:0:1"));
-  EXPECT_FALSE(IsLocalhost("::1:1"));
-  EXPECT_FALSE(IsLocalhost("0:0:0:0:0:0:0:0:1"));
-  EXPECT_FALSE(IsLocalhost("foo.localhost.com"));
-  EXPECT_FALSE(IsLocalhost("foo.localhoste"));
-}
-
-TEST(NetUtilTest, IsLocalhostTLD) {
-  EXPECT_TRUE(IsLocalhostTLD("foo.localhost"));
-  EXPECT_TRUE(IsLocalhostTLD("foo.localhost."));
-  EXPECT_FALSE(IsLocalhostTLD("foo.localhos"));
-  EXPECT_FALSE(IsLocalhostTLD("foo.localhost.com"));
-  EXPECT_FALSE(IsLocalhost("foo.localhoste"));
-}
-
-TEST(NetUtilTest, GoogleHost) {
-  struct GoogleHostCase {
-    GURL url;
-    bool expected_output;
-  };
-
-  const GoogleHostCase google_host_cases[] = {
-      {GURL("http://.google.com"), true},
-      {GURL("http://.youtube.com"), true},
-      {GURL("http://.gmail.com"), true},
-      {GURL("http://.doubleclick.net"), true},
-      {GURL("http://.gstatic.com"), true},
-      {GURL("http://.googlevideo.com"), true},
-      {GURL("http://.googleusercontent.com"), true},
-      {GURL("http://.googlesyndication.com"), true},
-      {GURL("http://.google-analytics.com"), true},
-      {GURL("http://.googleadservices.com"), true},
-      {GURL("http://.googleapis.com"), true},
-      {GURL("http://a.google.com"), true},
-      {GURL("http://b.youtube.com"), true},
-      {GURL("http://c.gmail.com"), true},
-      {GURL("http://google.com"), false},
-      {GURL("http://youtube.com"), false},
-      {GURL("http://gmail.com"), false},
-      {GURL("http://google.coma"), false},
-      {GURL("http://agoogle.com"), false},
-      {GURL("http://oogle.com"), false},
-      {GURL("http://google.co"), false},
-      {GURL("http://oggole.com"), false},
-  };
-
-  for (size_t i = 0; i < arraysize(google_host_cases); ++i) {
-    EXPECT_EQ(google_host_cases[i].expected_output,
-              HasGoogleHost(google_host_cases[i].url));
-  }
-}
-
 // Verify GetNetworkList().
 TEST(NetUtilTest, GetNetworkList) {
   NetworkInterfaceList list;
@@ -1149,8 +644,6 @@ TEST(NetUtilTest, GetWifiSSIDFromInterfaceList) {
 }
 #endif  // OS_LINUX
 
-namespace {
-
 #if defined(OS_WIN)
 bool read_int_or_bool(DWORD data_size,
                       PVOID data) {
@@ -1247,8 +740,6 @@ void TryChangeWifiOptions(int options) {
   EXPECT_EQ(previous_options, GetWifiOptions());
 }
 
-};  // namespace
-
 // Test SetWifiOptions().
 TEST(NetUtilTest, SetWifiOptionsTest) {
   TryChangeWifiOptions(0);
@@ -1258,83 +749,6 @@ TEST(NetUtilTest, SetWifiOptionsTest) {
                        WIFI_OPTIONS_MEDIA_STREAMING_MODE);
 }
 
-struct NonUniqueNameTestData {
-  bool is_unique;
-  const char* const hostname;
-};
-
-// Google Test pretty-printer.
-void PrintTo(const NonUniqueNameTestData& data, std::ostream* os) {
-  ASSERT_TRUE(data.hostname);
-  *os << " hostname: " << testing::PrintToString(data.hostname)
-      << "; is_unique: " << testing::PrintToString(data.is_unique);
-}
-
-const NonUniqueNameTestData kNonUniqueNameTestData[] = {
-    // Domains under ICANN-assigned domains.
-    { true, "google.com" },
-    { true, "google.co.uk" },
-    // Domains under private registries.
-    { true, "appspot.com" },
-    { true, "test.appspot.com" },
-    // Unreserved IPv4 addresses (in various forms).
-    { true, "8.8.8.8" },
-    { true, "99.64.0.0" },
-    { true, "212.15.0.0" },
-    { true, "212.15" },
-    { true, "212.15.0" },
-    { true, "3557752832" },
-    // Reserved IPv4 addresses (in various forms).
-    { false, "192.168.0.0" },
-    { false, "192.168.0.6" },
-    { false, "10.0.0.5" },
-    { false, "10.0" },
-    { false, "10.0.0" },
-    { false, "3232235526" },
-    // Unreserved IPv6 addresses.
-    { true, "FFC0:ba98:7654:3210:FEDC:BA98:7654:3210" },
-    { true, "2000:ba98:7654:2301:EFCD:BA98:7654:3210" },
-    // Reserved IPv6 addresses.
-    { false, "::192.9.5.5" },
-    { false, "FEED::BEEF" },
-    { false, "FEC0:ba98:7654:3210:FEDC:BA98:7654:3210" },
-    // 'internal'/non-IANA assigned domains.
-    { false, "intranet" },
-    { false, "intranet." },
-    { false, "intranet.example" },
-    { false, "host.intranet.example" },
-    // gTLDs under discussion, but not yet assigned.
-    { false, "intranet.corp" },
-    { false, "intranet.internal" },
-    // Invalid host names are treated as unique - but expected to be
-    // filtered out before then.
-    { true, "junk)(£)$*!@~#" },
-    { true, "w$w.example.com" },
-    { true, "nocolonsallowed:example" },
-    { true, "[::4.5.6.9]" },
-};
-
-class NetUtilNonUniqueNameTest
-    : public testing::TestWithParam<NonUniqueNameTestData> {
- public:
-  virtual ~NetUtilNonUniqueNameTest() {}
-
- protected:
-  bool IsUnique(const std::string& hostname) {
-    return !IsHostnameNonUnique(hostname);
-  }
-};
-
-// Test that internal/non-unique names are properly identified as such, but
-// that IP addresses and hosts beneath registry-controlled domains are flagged
-// as unique names.
-TEST_P(NetUtilNonUniqueNameTest, IsHostnameNonUnique) {
-  const NonUniqueNameTestData& test_data = GetParam();
-
-  EXPECT_EQ(test_data.is_unique, IsUnique(test_data.hostname));
-}
-
-INSTANTIATE_TEST_CASE_P(, NetUtilNonUniqueNameTest,
-                        testing::ValuesIn(kNonUniqueNameTestData));
+}  // namespace
 
 }  // namespace net