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

Unified Diff: net/http/http_stream_factory_impl_request.h

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.h
diff --git a/net/http/http_stream_factory_impl_request.h b/net/http/http_stream_factory_impl_request.h
index fd786f25c7549c554cf5d0d37e718101284c106e..e73f624779bbf50aee180f1e7685e0a79e232546 100644
--- a/net/http/http_stream_factory_impl_request.h
+++ b/net/http/http_stream_factory_impl_request.h
@@ -60,6 +60,11 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
const base::WeakPtr<SpdySession>& spdy_session,
bool direct);
+ // Called by an attached Job to record connection attempts made by the socket
+ // layer for this stream request.
+ void AddConnectionAttempts(
+ const ClientSocketHandle::ConnectionAttempts& attempts);
+
WebSocketHandshakeStreamBase::CreateHelper*
websocket_handshake_stream_create_helper() {
return websocket_handshake_stream_create_helper_;
@@ -104,6 +109,8 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
bool was_npn_negotiated() const override;
NextProto protocol_negotiated() const override;
bool using_spdy() const override;
+ const ClientSocketHandle::ConnectionAttempts& connection_attempts()
+ const override;
private:
// Used to orphan all jobs in |jobs_| other than |job| which becomes "bound"
@@ -133,6 +140,7 @@ class HttpStreamFactoryImpl::Request : public HttpStreamRequest {
// Protocol negotiated with the server.
NextProto protocol_negotiated_;
bool using_spdy_;
+ ClientSocketHandle::ConnectionAttempts connection_attempts_;
DISALLOW_COPY_AND_ASSIGN(Request);
};

Powered by Google App Engine
This is Rietveld 408576698