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

Unified Diff: net/quic/core/quic_session_test.cc

Issue 2591143003: Add QuicStrCat. (Closed)
Patch Set: correct quic_client_bin.cc Created 4 years 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/quic/core/quic_session_test.cc
diff --git a/net/quic/core/quic_session_test.cc b/net/quic/core/quic_session_test.cc
index 660e2d88a598503dd4d5b1874d9caa8515126c38..19d35de4e034b52eb838c5b13f9b145a7c9f3f1e 100644
--- a/net/quic/core/quic_session_test.cc
+++ b/net/quic/core/quic_session_test.cc
@@ -921,9 +921,8 @@ TEST_P(QuicSessionTestServer,
while (!headers_stream->flow_controller()->IsBlocked() && stream_id < 2000) {
EXPECT_FALSE(session_.IsConnectionFlowControlBlocked());
EXPECT_FALSE(session_.IsStreamFlowControlBlocked());
- headers["header"] = base::Uint64ToString(base::RandUint64()) +
- base::Uint64ToString(base::RandUint64()) +
- base::Uint64ToString(base::RandUint64());
+ headers["header"] = QuicStrCat("", base::RandUint64(), base::RandUint64(),
+ base::RandUint64());
headers_stream->WriteHeaders(stream_id, headers.Clone(), true, 0, nullptr);
stream_id += 2;
}

Powered by Google App Engine
This is Rietveld 408576698