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

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

Issue 2322233004: Landing Recent QUIC changes until Sun Sep 4 03:41:00 (Closed)
Patch Set: Remove simulation files from the build. Created 4 years, 3 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 14b73bf1a7abdf7b8b3ea002795815c10ed88ecf..12215118ec3df70fb84caa1e0e671a3cc9faa8be 100644
--- a/net/quic/core/quic_protocol.cc
+++ b/net/quic/core/quic_protocol.cc
@@ -188,7 +188,11 @@ QuicVersionVector FilterSupportedVersions(QuicVersionVector versions) {
filtered_versions.clear(); // Guaranteed by spec not to change capacity.
for (QuicVersion version : versions) {
if (version < QUIC_VERSION_32) {
- if (!FLAGS_quic_disable_pre_32) {
+ if (!FLAGS_quic_disable_pre_32 && !FLAGS_quic_disable_pre_34) {
+ filtered_versions.push_back(version);
+ }
+ } else if (version < QUIC_VERSION_34) {
+ if (!FLAGS_quic_disable_pre_34) {
filtered_versions.push_back(version);
}
} else if (version == QUIC_VERSION_35) {
@@ -780,6 +784,7 @@ StringPiece QuicPacket::Plaintext(QuicVersion version) const {
QuicVersionManager::QuicVersionManager(QuicVersionVector supported_versions)
: disable_pre_32_(FLAGS_quic_disable_pre_32),
+ disable_pre_34_(FLAGS_quic_disable_pre_34),
enable_version_35_(FLAGS_quic_enable_version_35),
enable_version_36_(FLAGS_quic_enable_version_36_v2),
allowed_supported_versions_(supported_versions),
@@ -790,9 +795,11 @@ QuicVersionManager::~QuicVersionManager() {}
const QuicVersionVector& QuicVersionManager::GetSupportedVersions() {
if (disable_pre_32_ != FLAGS_quic_disable_pre_32 ||
+ disable_pre_34_ != FLAGS_quic_disable_pre_34 ||
enable_version_35_ != FLAGS_quic_enable_version_35 ||
enable_version_36_ != FLAGS_quic_enable_version_36_v2) {
disable_pre_32_ = FLAGS_quic_disable_pre_32;
+ disable_pre_34_ = FLAGS_quic_disable_pre_34;
enable_version_35_ = FLAGS_quic_enable_version_35;
enable_version_36_ = FLAGS_quic_enable_version_36_v2;
filtered_supported_versions_ =
« 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