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

Unified Diff: net/quic/chromium/quic_chromium_client_stream_test.cc

Issue 2854833005: QUIC - stream id refactor for tests. (Closed)
Patch Set: rebase Created 3 years, 8 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/quic/chromium/quic_chromium_client_stream_test.cc
diff --git a/net/quic/chromium/quic_chromium_client_stream_test.cc b/net/quic/chromium/quic_chromium_client_stream_test.cc
index 95f258d9515acb06408423c5ee908e6d2da50310..89cba5f21028aaecc2a20c0cf65c148379c2ae16 100644
--- a/net/quic/chromium/quic_chromium_client_stream_test.cc
+++ b/net/quic/chromium/quic_chromium_client_stream_test.cc
@@ -17,6 +17,7 @@
#include "net/quic/core/quic_utils.h"
#include "net/quic/core/spdy_utils.h"
#include "net/quic/test_tools/crypto_test_utils.h"
+#include "net/quic/test_tools/quic_spdy_session_peer.h"
#include "net/quic/test_tools/quic_test_utils.h"
#include "net/test/gtest_util.h"
#include "testing/gmock/include/gmock/gmock.h"
@@ -246,6 +247,14 @@ class QuicChromiumClientStreamTest
return h;
}
+ QuicStreamId GetNthClientInitiatedStreamId(int n) {
+ return QuicSpdySessionPeer::GetNthClientInitiatedStreamId(session_, n);
+ }
+
+ QuicStreamId GetNthServerInitiatedStreamId(int n) {
+ return QuicSpdySessionPeer::GetNthServerInitiatedStreamId(session_, n);
+ }
+
QuicCryptoClientConfig crypto_config_;
testing::StrictMock<MockDelegate> delegate_;
MockQuicConnectionHelper helper_;
@@ -593,7 +602,7 @@ TEST_P(QuicChromiumClientStreamTest, HeadersBeforeDelegate) {
// We don't use stream_ because we want an incoming server push
// stream.
QuicChromiumClientStream* stream = new QuicChromiumClientStream(
- kServerDataStreamId1, &session_, NetLogWithSource());
+ GetNthServerInitiatedStreamId(0), &session_, NetLogWithSource());
session_.ActivateStream(base::WrapUnique(stream));
InitializeHeaders();

Powered by Google App Engine
This is Rietveld 408576698