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

Unified Diff: net/http/http_stream_factory_impl_request.cc

Issue 1006643002: Plumb connection attempts from (non-proxy) ConnectJobs to HttpNetworkTransaction. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Plumb up to HttpNetworkTransaction 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
Index: net/http/http_stream_factory_impl_request.cc
diff --git a/net/http/http_stream_factory_impl_request.cc b/net/http/http_stream_factory_impl_request.cc
index c4d1d077280b69f7edda427b7bc0098f1307e591..fdd4157731dc0aeda424f37ec9e75bcc769d1568 100644
--- a/net/http/http_stream_factory_impl_request.cc
+++ b/net/http/http_stream_factory_impl_request.cc
@@ -234,6 +234,11 @@ bool HttpStreamFactoryImpl::Request::using_spdy() const {
return using_spdy_;
}
+const ClientSocketHandle::ConnectionAttempts&
+HttpStreamFactoryImpl::Request::connection_attempts() const {
+ return connection_attempts_;
+}
+
void
HttpStreamFactoryImpl::Request::RemoveRequestFromSpdySessionRequestMap() {
if (spdy_session_key_.get()) {
@@ -315,6 +320,12 @@ void HttpStreamFactoryImpl::Request::OnNewSpdySessionReady(
}
}
+void HttpStreamFactoryImpl::Request::AddConnectionAttempts(
+ const ClientSocketHandle::ConnectionAttempts& attempts) {
+ for (const auto& attempt : attempts)
+ connection_attempts_.push_back(attempt);
+}
+
void HttpStreamFactoryImpl::Request::OrphanJobsExcept(Job* job) {
DCHECK(job);
DCHECK(!bound_job_.get());

Powered by Google App Engine
This is Rietveld 408576698