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

Unified Diff: trunk/src/net/quic/quic_protocol.cc

Issue 16374004: Revert 204046 "Land Recent QUIC changes." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 6 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 | « trunk/src/net/quic/quic_protocol.h ('k') | trunk/src/net/quic/quic_session.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/net/quic/quic_protocol.cc
===================================================================
--- trunk/src/net/quic/quic_protocol.cc (revision 204061)
+++ trunk/src/net/quic/quic_protocol.cc (working copy)
@@ -16,39 +16,31 @@
size_t GetPacketHeaderSize(QuicPacketHeader header) {
return GetPacketHeaderSize(header.public_header.guid_length,
header.public_header.version_flag,
- header.public_header.sequence_number_length,
header.is_in_fec_group);
}
size_t GetPacketHeaderSize(QuicGuidLength guid_length,
bool include_version,
- QuicSequenceNumberLength sequence_number_length,
InFecGroup is_in_fec_group) {
return kPublicFlagsSize + guid_length +
- (include_version ? kQuicVersionSize : 0) + sequence_number_length +
+ (include_version ? kQuicVersionSize : 0) + kSequenceNumberSize +
kPrivateFlagsSize + (is_in_fec_group == IN_FEC_GROUP ? kFecGroupSize : 0);
}
size_t GetPublicResetPacketSize() {
return kPublicFlagsSize + PACKET_8BYTE_GUID + kPublicResetNonceSize +
- PACKET_6BYTE_SEQUENCE_NUMBER;
+ kSequenceNumberSize;
}
-size_t GetStartOfFecProtectedData(
- QuicGuidLength guid_length,
- bool include_version,
- QuicSequenceNumberLength sequence_number_length) {
- return GetPacketHeaderSize(
- guid_length, include_version, sequence_number_length, IN_FEC_GROUP);
+size_t GetStartOfFecProtectedData(QuicGuidLength guid_length,
+ bool include_version) {
+ return GetPacketHeaderSize(guid_length, include_version, IN_FEC_GROUP);
}
-size_t GetStartOfEncryptedData(
- QuicGuidLength guid_length,
- bool include_version,
- QuicSequenceNumberLength sequence_number_length) {
+size_t GetStartOfEncryptedData(QuicGuidLength guid_length,
+ bool include_version) {
// Don't include the fec size, since encryption starts before private flags.
- return GetPacketHeaderSize(
- guid_length, include_version, sequence_number_length, NOT_IN_FEC_GROUP) -
+ return GetPacketHeaderSize(guid_length, include_version, NOT_IN_FEC_GROUP) -
kPrivateFlagsSize;
}
@@ -63,8 +55,7 @@
: guid(0),
guid_length(PACKET_8BYTE_GUID),
reset_flag(false),
- version_flag(false),
- sequence_number_length(PACKET_6BYTE_SEQUENCE_NUMBER) {
+ version_flag(false) {
}
QuicPacketPublicHeader::QuicPacketPublicHeader(
@@ -73,7 +64,6 @@
guid_length(other.guid_length),
reset_flag(other.reset_flag),
version_flag(other.version_flag),
- sequence_number_length(other.sequence_number_length),
versions(other.versions) {
}
@@ -261,6 +251,16 @@
QuicFecData::QuicFecData() {}
+bool QuicFecData::operator==(const QuicFecData& other) const {
+ if (fec_group != other.fec_group) {
+ return false;
+ }
+ if (redundancy != other.redundancy) {
+ return false;
+ }
+ return true;
+}
+
QuicData::~QuicData() {
if (owns_buffer_) {
delete [] const_cast<char*>(buffer_);
@@ -268,29 +268,25 @@
}
StringPiece QuicPacket::FecProtectedData() const {
- const size_t start_of_fec = GetStartOfFecProtectedData(
- guid_length_, includes_version_, sequence_number_length_);
+ const size_t start_of_fec = GetStartOfFecProtectedData(guid_length_,
+ includes_version_);
return StringPiece(data() + start_of_fec, length() - start_of_fec);
}
StringPiece QuicPacket::AssociatedData() const {
- return StringPiece(
- data() + kStartOfHashData,
- GetStartOfEncryptedData(
- guid_length_, includes_version_, sequence_number_length_) -
- kStartOfHashData);
+ return StringPiece(data() + kStartOfHashData,
+ GetStartOfEncryptedData(guid_length_, includes_version_) -
+ kStartOfHashData);
}
StringPiece QuicPacket::BeforePlaintext() const {
return StringPiece(data(), GetStartOfEncryptedData(guid_length_,
- includes_version_,
- sequence_number_length_));
+ includes_version_));
}
StringPiece QuicPacket::Plaintext() const {
const size_t start_of_encrypted_data =
- GetStartOfEncryptedData(
- guid_length_, includes_version_, sequence_number_length_);
+ GetStartOfEncryptedData(guid_length_, includes_version_);
return StringPiece(data() + start_of_encrypted_data,
length() - start_of_encrypted_data);
}
« no previous file with comments | « trunk/src/net/quic/quic_protocol.h ('k') | trunk/src/net/quic/quic_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698