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

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

Issue 2778943002: Pass HttpServerProperties in to QuicHttpStream so that it can mark QUIC broken. (Closed)
Patch Set: Rebase Created 3 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/chromium/quic_stream_factory.h ('k') | net/quic/chromium/quic_stream_factory_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/chromium/quic_stream_factory.cc
diff --git a/net/quic/chromium/quic_stream_factory.cc b/net/quic/chromium/quic_stream_factory.cc
index ffcabf5ae2491738d32f831c90dee65762cd9a4c..f7e7fcd9cd46d0b8e4a815f7611b8061c67014a3 100644
--- a/net/quic/chromium/quic_stream_factory.cc
+++ b/net/quic/chromium/quic_stream_factory.cc
@@ -663,8 +663,10 @@ int QuicStreamFactory::Job::DoConnectComplete(int rv) {
return OK;
}
-QuicStreamRequest::QuicStreamRequest(QuicStreamFactory* factory)
- : factory_(factory) {}
+QuicStreamRequest::QuicStreamRequest(
+ QuicStreamFactory* factory,
+ HttpServerProperties* http_server_properties)
+ : factory_(factory), http_server_properties_(http_server_properties) {}
QuicStreamRequest::~QuicStreamRequest() {
if (factory_ && !callback_.is_null())
@@ -714,7 +716,7 @@ base::TimeDelta QuicStreamRequest::GetTimeDelayForWaitingJob() const {
std::unique_ptr<QuicHttpStream> QuicStreamRequest::CreateStream() {
if (!session_)
return nullptr;
- return base::MakeUnique<QuicHttpStream>(session_);
+ return base::MakeUnique<QuicHttpStream>(session_, http_server_properties_);
}
std::unique_ptr<BidirectionalStreamImpl>
@@ -1190,7 +1192,7 @@ void QuicStreamFactory::OnCertVerifyJobComplete(CertVerifierJob* job, int rv) {
std::unique_ptr<QuicHttpStream> QuicStreamFactory::CreateFromSession(
QuicChromiumClientSession* session) {
return std::unique_ptr<QuicHttpStream>(
- new QuicHttpStream(session->GetWeakPtr()));
+ new QuicHttpStream(session->GetWeakPtr(), http_server_properties_));
}
bool QuicStreamFactory::IsQuicDisabled() const {
« no previous file with comments | « net/quic/chromium/quic_stream_factory.h ('k') | net/quic/chromium/quic_stream_factory_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698