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

Unified Diff: net/http/http_response_info.h

Issue 2461683002: Slice enum HttpResponseInfo::ConnectionInfo by QUIC version. (Closed)
Patch Set: Update recently introduced CONNECTION_INFO_QUIC occurrences. Created 4 years, 1 month 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 | « content/browser/frame_host/navigation_handle_impl_unittest.cc ('k') | net/http/http_response_info.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_response_info.h
diff --git a/net/http/http_response_info.h b/net/http/http_response_info.h
index dbfb5f68bcc8634c9b85978911b70e78a598d0a0..83bc630bfa64cbcde60b2f52e5cc7d196d932f5b 100644
--- a/net/http/http_response_info.h
+++ b/net/http/http_response_info.h
@@ -37,11 +37,16 @@ class NET_EXPORT HttpResponseInfo {
CONNECTION_INFO_DEPRECATED_SPDY2 = 2,
CONNECTION_INFO_DEPRECATED_SPDY3 = 3,
CONNECTION_INFO_HTTP2 = 4, // HTTP/2.
- CONNECTION_INFO_QUIC = 5,
+ CONNECTION_INFO_QUIC_UNKNOWN_VERSION = 5,
CONNECTION_INFO_DEPRECATED_HTTP2_14 = 6, // HTTP/2 draft-14.
CONNECTION_INFO_DEPRECATED_HTTP2_15 = 7, // HTTP/2 draft-15.
CONNECTION_INFO_HTTP0_9 = 8,
CONNECTION_INFO_HTTP1_0 = 9,
+ CONNECTION_INFO_QUIC_32 = 10,
+ CONNECTION_INFO_QUIC_33 = 11,
+ CONNECTION_INFO_QUIC_34 = 12,
+ CONNECTION_INFO_QUIC_35 = 13,
+ CONNECTION_INFO_QUIC_36 = 14,
NUM_OF_CONNECTION_INFOS,
};
@@ -89,7 +94,7 @@ class NET_EXPORT HttpResponseInfo {
bool response_truncated) const;
// Whether QUIC is used or not.
- bool DidUseQuic() const { return connection_info == CONNECTION_INFO_QUIC; }
+ bool DidUseQuic() const;
// The following is only defined if the request_time member is set.
// If this resource was found in the cache, then this bool is set, and
« no previous file with comments | « content/browser/frame_host/navigation_handle_impl_unittest.cc ('k') | net/http/http_response_info.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698