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

Unified Diff: net/dns/address_sorter_posix_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/dns/address_sorter_posix_unittest.cc
diff --git a/net/dns/address_sorter_posix_unittest.cc b/net/dns/address_sorter_posix_unittest.cc
index 13778ebefeeb458869e56f996c0110ff99126e36..d40dfbf25ae6d818863ac46a2e9df65bf699e170 100644
--- a/net/dns/address_sorter_posix_unittest.cc
+++ b/net/dns/address_sorter_posix_unittest.cc
@@ -73,7 +73,7 @@ class TestUDPClientSocket : public DatagramClientSocket {
int Connect(const IPEndPoint& remote) override {
if (connected_)
return ERR_UNEXPECTED;
- AddressMapping::const_iterator it = mapping_->find(remote.address());
+ AddressMapping::const_iterator it = mapping_->find(remote.address_number());
if (it == mapping_->end())
return ERR_FAILED;
connected_ = true;
@@ -179,10 +179,10 @@ class AddressSorterPosixTest : public testing::Test {
for (size_t i = 0; (i < result.size()) || (order[i] >= 0); ++i) {
IPEndPoint expected = order[i] >= 0 ? list[order[i]] : IPEndPoint();
IPEndPoint actual = i < result.size() ? result[i] : IPEndPoint();
- EXPECT_TRUE(expected.address() == actual.address()) <<
- "Address out of order at position " << i << "\n" <<
- " Actual: " << actual.ToStringWithoutPort() << "\n" <<
- "Expected: " << expected.ToStringWithoutPort();
+ EXPECT_TRUE(expected.address_number() == actual.address_number())
+ << "Address out of order at position " << i << "\n"
+ << " Actual: " << actual.ToStringWithoutPort() << "\n"
+ << "Expected: " << expected.ToStringWithoutPort();
}
}

Powered by Google App Engine
This is Rietveld 408576698