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

Unified Diff: net/http/http_network_transaction_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: Feedback eroman 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/http/http_network_transaction_unittest.cc
diff --git a/net/http/http_network_transaction_unittest.cc b/net/http/http_network_transaction_unittest.cc
index ed7470b3a618ffba8be29837496992d3e2f5ca21..2caceac750587ecfee608ea32c2f791906a3a4fb 100644
--- a/net/http/http_network_transaction_unittest.cc
+++ b/net/http/http_network_transaction_unittest.cc
@@ -397,7 +397,7 @@ class HttpNetworkTransactionTest
bool got_endpoint =
trans->GetRemoteEndpoint(&out.remote_endpoint_after_start);
EXPECT_EQ(got_endpoint,
- out.remote_endpoint_after_start.address().size() > 0);
+ out.remote_endpoint_after_start.address().bytes().size() > 0);
rv = ReadTransaction(trans.get(), &out.response_data);
EXPECT_EQ(OK, rv);
@@ -1645,7 +1645,7 @@ TEST_P(HttpNetworkTransactionTest, NonKeepAliveConnectionReset) {
IPEndPoint endpoint;
EXPECT_TRUE(trans->GetRemoteEndpoint(&endpoint));
- EXPECT_LT(0u, endpoint.address().size());
+ EXPECT_LT(0u, endpoint.address().bytes().size());
}
// What do various browsers do when the server closes a non-keepalive
@@ -9061,7 +9061,7 @@ TEST_P(HttpNetworkTransactionTest, RequestWriteError) {
IPEndPoint endpoint;
EXPECT_TRUE(trans->GetRemoteEndpoint(&endpoint));
- EXPECT_LT(0u, endpoint.address().size());
+ EXPECT_LT(0u, endpoint.address().bytes().size());
eroman 2016/01/13 23:19:42 same here
martijnc 2016/01/14 22:48:17 Done.
}
// Check that a connection closed after the start of the headers finishes ok.
@@ -9104,7 +9104,7 @@ TEST_P(HttpNetworkTransactionTest, ConnectionClosedAfterStartOfHeaders) {
IPEndPoint endpoint;
EXPECT_TRUE(trans->GetRemoteEndpoint(&endpoint));
- EXPECT_LT(0u, endpoint.address().size());
+ EXPECT_LT(0u, endpoint.address().bytes().size());
}
// Make sure that a dropped connection while draining the body for auth

Powered by Google App Engine
This is Rietveld 408576698