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

Unified Diff: net/quic/core/quic_protocol.cc

Issue 2236463004: Rename QuicSupportedVersions to QuicAllSupportedVersions. Add QuicCurrentSupportedVersions which re… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@129437595
Patch Set: update more files outside net/ Created 4 years, 4 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/core/quic_protocol.h ('k') | net/quic/core/quic_protocol_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/quic_protocol.cc
diff --git a/net/quic/core/quic_protocol.cc b/net/quic/core/quic_protocol.cc
index aff9d26831f6d1afe988600e9ac093cd3e0ee929..dd908d1ff6f9b611fe7f1f1e53b8a0c1fa13dc31 100644
--- a/net/quic/core/quic_protocol.cc
+++ b/net/quic/core/quic_protocol.cc
@@ -171,7 +171,7 @@ bool ContainsQuicTag(const QuicTagVector& tag_vector, QuicTag tag) {
tag_vector.end();
}
-QuicVersionVector QuicSupportedVersions() {
+QuicVersionVector AllSupportedVersions() {
QuicVersionVector supported_versions;
for (size_t i = 0; i < arraysize(kSupportedQuicVersions); ++i) {
supported_versions.push_back(kSupportedQuicVersions[i]);
@@ -179,6 +179,10 @@ QuicVersionVector QuicSupportedVersions() {
return supported_versions;
}
+QuicVersionVector CurrentSupportedVersions() {
+ return FilterSupportedVersions(AllSupportedVersions());
+}
+
QuicVersionVector FilterSupportedVersions(QuicVersionVector versions) {
QuicVersionVector filtered_versions(versions.size());
filtered_versions.clear(); // Guaranteed by spec not to change capacity.
@@ -844,12 +848,12 @@ StringPiece QuicPacket::Plaintext(QuicVersion version) const {
length() - start_of_encrypted_data);
}
-QuicVersionManager::QuicVersionManager(QuicVersionVector supported_versions) {
- enable_quic_version_35_ = FLAGS_quic_enable_version_35;
- enable_quic_version_36_ = FLAGS_quic_enable_version_36;
- allowed_supported_versions_ = supported_versions;
- filtered_supported_versions_ = FilterSupportedVersions(supported_versions);
-}
+QuicVersionManager::QuicVersionManager(QuicVersionVector supported_versions)
+ : enable_quic_version_35_(FLAGS_quic_enable_version_35),
+ enable_quic_version_36_(FLAGS_quic_enable_version_36),
+ allowed_supported_versions_(supported_versions),
+ filtered_supported_versions_(
+ FilterSupportedVersions(supported_versions)) {}
const QuicVersionVector& QuicVersionManager::GetSupportedVersions() {
if (enable_quic_version_35_ != FLAGS_quic_enable_version_35 ||
« no previous file with comments | « net/quic/core/quic_protocol.h ('k') | net/quic/core/quic_protocol_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698