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

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

Issue 2856243003: Revert of Landing Recent QUIC changes until Sat Apr 29 00:22:04 2017 +0000 (Closed)
Patch Set: Created 3 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 | « net/quic/chromium/quic_chromium_client_session_test.cc ('k') | net/quic/chromium/quic_http_stream_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 764c2f408f1d9732f31852936fa9a6307235cfd4..95f258d9515acb06408423c5ee908e6d2da50310 100644
--- a/net/quic/chromium/quic_chromium_client_stream_test.cc
+++ b/net/quic/chromium/quic_chromium_client_stream_test.cc
@@ -16,12 +16,9 @@
#include "net/quic/core/quic_client_session_base.h"
#include "net/quic/core/quic_utils.h"
#include "net/quic/core/spdy_utils.h"
-#include "net/quic/platform/api/quic_ptr_util.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 "net/tools/quic/quic_spdy_client_stream.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gmock_mutant.h"
@@ -61,14 +58,6 @@
private:
DISALLOW_COPY_AND_ASSIGN(MockDelegate);
-};
-
-class TestQuicClientSessionBaseStream : public QuicSpdyStream {
- public:
- TestQuicClientSessionBaseStream(QuicStreamId id, QuicSpdySession* session)
- : QuicSpdyStream(id, session) {}
-
- void OnDataAvailable() override {}
};
class MockQuicClientSessionBase : public QuicClientSessionBase {
@@ -139,10 +128,6 @@
const QuicReferenceCountedPointer<QuicAckListenerInterface>&
ack_listener));
MOCK_METHOD1(OnHeadersHeadOfLineBlocking, void(QuicTime::Delta delta));
-
- std::unique_ptr<QuicStream> CreateStream(QuicStreamId id) {
- return QuicMakeUnique<TestQuicClientSessionBaseStream>(id, this);
- }
using QuicSession::ActivateStream;
@@ -261,14 +246,6 @@
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_;
@@ -616,7 +593,7 @@
// We don't use stream_ because we want an incoming server push
// stream.
QuicChromiumClientStream* stream = new QuicChromiumClientStream(
- GetNthServerInitiatedStreamId(0), &session_, NetLogWithSource());
+ kServerDataStreamId1, &session_, NetLogWithSource());
session_.ActivateStream(base::WrapUnique(stream));
InitializeHeaders();
« no previous file with comments | « net/quic/chromium/quic_chromium_client_session_test.cc ('k') | net/quic/chromium/quic_http_stream_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698