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

Unified Diff: net/quic/p2p/quic_p2p_session.cc

Issue 1957753002: Clean up IPAddress::ToString related TODO. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/p2p/quic_p2p_session.cc
diff --git a/net/quic/p2p/quic_p2p_session.cc b/net/quic/p2p/quic_p2p_session.cc
index 94777b307d5a69edb548716eb1110b02c5b1c8ba..17825c0af6188b4c0b5be482edb830ce962e13f6 100644
--- a/net/quic/p2p/quic_p2p_session.cc
+++ b/net/quic/p2p/quic_p2p_session.cc
@@ -26,12 +26,6 @@ QuicP2PSession::QuicP2PSession(const QuicConfig& config,
crypto_stream_(new QuicP2PCryptoStream(this, crypto_config)),
read_buffer_(new IOBuffer(static_cast<size_t>(kMaxPacketSize))) {
DCHECK(config.negotiated());
-
- // Non-null IP address needs to be passed here because QuicConnection uses
- // ToString() to format addresses for logging and ToString() is not allowed
- // for empty addresses.
- // TODO(sergeyu): Fix QuicConnection and remove SetSelfAddress() call below.
- this->connection()->SetSelfAddress(IPEndPoint(IPAddress::IPv4AllZeros(), 0));
}
QuicP2PSession::~QuicP2PSession() {}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698