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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: blimp/net/tcp_transport_unittest.cc

Issue 2439403003: Refactor BlimpConnection to TCPConnection (Closed)
Patch Set: Added missing Engine Transport Created 4 years, 2 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
« blimp/net/tcp_engine_transport.cc ('K') | « blimp/net/tcp_engine_transport.cc ('k') | blimp/net/test_common.h » ('j') | build/args/blimp_linux_client.gn » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/net/tcp_transport_unittest.cc
diff --git a/blimp/net/tcp_transport_unittest.cc b/blimp/net/tcp_transport_unittest.cc
index 97ead2f9cfbb49ce049e87765a5d8cc529f8e34e..0a80efefd0ac88074d05a63c36304a917b7ed4e9 100644
--- a/blimp/net/tcp_transport_unittest.cc
+++ b/blimp/net/tcp_transport_unittest.cc
@@ -44,7 +44,7 @@ class TCPTransportTest : public testing::Test {
net::IPEndPoint GetLocalEndpoint() const {
net::IPEndPoint local_address;
- CHECK_EQ(net::OK, engine_.GetLocalAddress(&local_address));
+ engine_.GetLocalAddress(&local_address);
Wez 2016/11/09 22:47:17 Why remove the CHECK?
return local_address;
}
« blimp/net/tcp_engine_transport.cc ('K') | « blimp/net/tcp_engine_transport.cc ('k') | blimp/net/test_common.h » ('j') | build/args/blimp_linux_client.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698