Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1196)

Unified Diff: net/base/ip_endpoint_unittest.cc

Issue 1565303002: Change IPEndpoint::address() to return a net::IPAddress (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Android Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/base/ip_endpoint_unittest.cc
diff --git a/net/base/ip_endpoint_unittest.cc b/net/base/ip_endpoint_unittest.cc
index 4da0bbe6290788f09b88798dc1468436fac167da..2676dc532c631aec4a832aac5f0b2e0acb4e1b97 100644
--- a/net/base/ip_endpoint_unittest.cc
+++ b/net/base/ip_endpoint_unittest.cc
@@ -81,7 +81,7 @@ TEST_F(IPEndPointTest, Constructor) {
for (uint16_t index = 0; index < test_count; ++index) {
IPEndPoint endpoint(tests[index].ip_address, 80);
EXPECT_EQ(80, endpoint.port());
- EXPECT_EQ(tests[index].ip_address, endpoint.address());
+ EXPECT_EQ(tests[index].ip_address, endpoint.address_number());
}
}
@@ -91,7 +91,7 @@ TEST_F(IPEndPointTest, Assignment) {
IPEndPoint dest = src;
EXPECT_EQ(src.port(), dest.port());
- EXPECT_EQ(src.address(), dest.address());
+ EXPECT_EQ(src.address_number(), dest.address_number());
}
}
@@ -101,7 +101,7 @@ TEST_F(IPEndPointTest, Copy) {
IPEndPoint dest(src);
EXPECT_EQ(src.port(), dest.port());
- EXPECT_EQ(src.address(), dest.address());
+ EXPECT_EQ(src.address_number(), dest.address_number());
}
}
@@ -124,7 +124,7 @@ TEST_F(IPEndPointTest, ToFromSockAddr) {
IPEndPoint ip_endpoint2;
EXPECT_TRUE(ip_endpoint2.FromSockAddr(storage.addr, storage.addr_len));
EXPECT_EQ(ip_endpoint.port(), ip_endpoint2.port());
- EXPECT_EQ(ip_endpoint.address(), ip_endpoint2.address());
+ EXPECT_EQ(ip_endpoint.address_number(), ip_endpoint2.address_number());
}
}

Powered by Google App Engine
This is Rietveld 408576698