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

Unified Diff: net/quic/quic_server_test.cc

Issue 1031243002: Unify the QUIC dispatcher and make the QuicServer work. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: QuicChromeServerDispatchPacketTest Created 5 years, 9 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/quic_server.cc ('k') | net/quic/test_tools/mock_quic_dispatcher.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_server_test.cc
diff --git a/net/quic/quic_server_test.cc b/net/quic/quic_server_test.cc
index 9e4e8944fc7c096b7cfef559d37346a6ffd9f72b..cac66b63b39ae35d9e3021f884e99d5148c55b2d 100644
--- a/net/quic/quic_server_test.cc
+++ b/net/quic/quic_server_test.cc
@@ -24,9 +24,9 @@ class QuicChromeServerDispatchPacketTest : public ::testing::Test {
: crypto_config_("blah", QuicRandom::GetInstance()),
dispatcher_(config_,
crypto_config_,
- new QuicDispatcher::DefaultPacketWriterFactory(),
- &helper_) {
- dispatcher_.Initialize(nullptr);
+ new tools::QuicDispatcher::DefaultPacketWriterFactory(),
+ new MockHelper) {
+ dispatcher_.InitializeWithWriter(nullptr);
}
void DispatchPacket(const QuicEncryptedPacket& packet) {
@@ -37,7 +37,6 @@ class QuicChromeServerDispatchPacketTest : public ::testing::Test {
protected:
QuicConfig config_;
QuicCryptoServerConfig crypto_config_;
- MockHelper helper_;
MockQuicDispatcher dispatcher_;
};
« no previous file with comments | « net/quic/quic_server.cc ('k') | net/quic/test_tools/mock_quic_dispatcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698