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

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

Issue 2403193003: Landing Recent QUIC changes until 9:41 AM, Oct 10, 2016 UTC-7 (Closed)
Patch Set: git cl format Created 4 years, 2 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_packet_creator.cc ('k') | net/quic/core/quic_protocol.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/quic_protocol.h
diff --git a/net/quic/core/quic_protocol.h b/net/quic/core/quic_protocol.h
index ce9fd2591bb1bef1ad344decfced4bbef34428cc..bc609d2e836ed9e5850535f4c4e912d1709e355f 100644
--- a/net/quic/core/quic_protocol.h
+++ b/net/quic/core/quic_protocol.h
@@ -56,9 +56,6 @@ typedef uint16_t QuicPacketLength;
const QuicByteCount kDefaultMaxPacketSize = 1350;
// Default initial maximum size in bytes of a QUIC packet for servers.
const QuicByteCount kDefaultServerMaxPacketSize = 1000;
-// Minimum size of a QUIC packet, used if a server receives packets from a
-// client with unusual network headers. 1280 - sizeof(eth) - sizeof(ipv6).
-const QuicByteCount kMinimumSupportedPacketSize = 1214;
// The maximum packet size of any QUIC packet, based on ethernet's max size,
// minus the IP and UDP headers. IPv6 has a 40 byte header, UDP adds an
// additional 8 bytes. This is a total overhead of 48 bytes. Ethernet's
@@ -345,17 +342,8 @@ enum QuicPacketPrivateFlags {
// Bit 0: Does this packet contain an entropy bit?
PACKET_PRIVATE_FLAGS_ENTROPY = 1 << 0,
- // Bit 1: Payload is part of an FEC group?
- PACKET_PRIVATE_FLAGS_FEC_GROUP = 1 << 1,
-
- // Bit 2: Payload is FEC as opposed to frames?
- PACKET_PRIVATE_FLAGS_FEC = 1 << 2,
-
- // All bits set (bits 3-7 are not currently used): 00000111
- PACKET_PRIVATE_FLAGS_MAX = (1 << 3) - 1,
-
- // For version 32 (bits 1-7 are not used): 00000001
- PACKET_PRIVATE_FLAGS_MAX_VERSION_32 = (1 << 1) - 1
+ // (bits 1-7 are not used): 00000001
+ PACKET_PRIVATE_FLAGS_MAX = (1 << 1) - 1
};
// The available versions of QUIC. Guaranteed that the integer value of the enum
@@ -368,8 +356,6 @@ enum QuicVersion {
// Special case to indicate unknown/unsupported QUIC version.
QUIC_VERSION_UNSUPPORTED = 0,
- QUIC_VERSION_30 = 30, // Add server side support of cert transparency.
- QUIC_VERSION_31 = 31, // Adds a hash of the client hello to crypto proof.
QUIC_VERSION_32 = 32, // FEC related fields are removed from wire format.
QUIC_VERSION_33 = 33, // Adds diversification nonces.
QUIC_VERSION_34 = 34, // Deprecates entropy, removes private flag from packet
@@ -390,7 +376,7 @@ enum QuicVersion {
// http://sites/quic/adding-and-removing-versions
static const QuicVersion kSupportedQuicVersions[] = {
QUIC_VERSION_36, QUIC_VERSION_35, QUIC_VERSION_34, QUIC_VERSION_33,
- QUIC_VERSION_32, QUIC_VERSION_31, QUIC_VERSION_30};
+ QUIC_VERSION_32};
typedef std::vector<QuicVersion> QuicVersionVector;
@@ -774,7 +760,6 @@ struct NET_EXPORT_PRIVATE QuicPacketHeader {
QuicPathId path_id;
bool entropy_flag;
QuicPacketEntropyHash entropy_hash;
- bool fec_flag;
};
struct NET_EXPORT_PRIVATE QuicPublicResetPacket {
@@ -783,6 +768,8 @@ struct NET_EXPORT_PRIVATE QuicPublicResetPacket {
QuicPacketPublicHeader public_header;
QuicPublicResetNonceProof nonce_proof;
+ // TODO(fayang): remove rejected_packet_number when deprecating
+ // FLAGS_quic_remove_packet_number_from_public_reset.
QuicPacketNumber rejected_packet_number;
IPEndPoint client_address;
};
@@ -1072,6 +1059,7 @@ enum LossDetectionType {
kNack, // Used to mimic TCP's loss detection.
kTime, // Time based loss detection.
kAdaptiveTime, // Adaptive time based loss detection.
+ kLazyFack, // Nack based but with FACK disabled for the first ack.
};
struct NET_EXPORT_PRIVATE QuicRstStreamFrame {
@@ -1331,7 +1319,6 @@ class NET_EXPORT_PRIVATE QuicReceivedPacket : public QuicEncryptedPacket {
size_t length,
QuicTime receipt_time,
bool owns_buffer,
- bool potentially_small_mtu,
int ttl,
bool ttl_valid);
@@ -1344,8 +1331,6 @@ class NET_EXPORT_PRIVATE QuicReceivedPacket : public QuicEncryptedPacket {
// This is the TTL of the packet, assuming ttl_vaild_ is true.
int ttl() const { return ttl_; }
- bool potentially_small_mtu() const { return potentially_small_mtu_; }
-
// By default, gtest prints the raw bytes of an object. The bool data
// member (in the base class QuicData) causes this object to have padding
// bytes, which causes the default gtest object printer to read
@@ -1357,7 +1342,6 @@ class NET_EXPORT_PRIVATE QuicReceivedPacket : public QuicEncryptedPacket {
private:
const QuicTime receipt_time_;
int ttl_;
- bool potentially_small_mtu_;
DISALLOW_COPY_AND_ASSIGN(QuicReceivedPacket);
};
@@ -1405,8 +1389,6 @@ class NET_EXPORT_PRIVATE QuicVersionManager {
const QuicVersionVector& GetSupportedVersions();
private:
- // FLAGS_quic_disable_pre_32
- bool disable_pre_32_;
// FLAGS_quic_disable_pre_34
bool disable_pre_34_;
// FLAGS_quic_enable_version_35
« no previous file with comments | « net/quic/core/quic_packet_creator.cc ('k') | net/quic/core/quic_protocol.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698