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

Unified Diff: net/http/http_stream_factory.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.h
diff --git a/net/http/http_stream_factory.h b/net/http/http_stream_factory.h
index e6712623fb0428d3f36a213dcaa31870b1c07379..379c5717d7d66640adceedffad7f35157d9ef43e 100644
--- a/net/http/http_stream_factory.h
+++ b/net/http/http_stream_factory.h
@@ -16,6 +16,7 @@
#include "net/base/net_export.h"
#include "net/base/request_priority.h"
#include "net/http/http_server_properties.h"
+#include "net/socket/client_socket_handle.h"
// This file can be included from net/http even though
// it is in net/websockets because it doesn't
// introduce any link dependency to net/websockets.
@@ -170,6 +171,10 @@ class NET_EXPORT_PRIVATE HttpStreamRequest {
// Returns true if this stream is being fetched over SPDY.
virtual bool using_spdy() const = 0;
+
+ // Returns socket-layer connection attempts made for this stream request.
+ virtual const ClientSocketHandle::ConnectionAttempts& connection_attempts()
Ryan Hamilton 2015/03/17 03:22:28 If this struct is going to be used outside of the
Deprecated (see juliatuttle) 2015/03/23 17:45:35 Yeah, I agree, but I'm not sure where. I suppose I
Ryan Hamilton 2015/03/24 19:20:19 My vote would go for net/socket/connection_attempt
+ const = 0;
};
// The HttpStreamFactory defines an interface for creating usable HttpStreams.

Powered by Google App Engine
This is Rietveld 408576698