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

Unified Diff: net/quic/quic_protocol.cc

Issue 862133002: Update from https://crrev.com/312398 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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
Index: net/quic/quic_protocol.cc
diff --git a/net/quic/quic_protocol.cc b/net/quic/quic_protocol.cc
index 1d5a312210b02be7f000da459690379c8f49b49d..56dfa97b999802b91f86b0c7b5bfa020ed43baa9 100644
--- a/net/quic/quic_protocol.cc
+++ b/net/quic/quic_protocol.cc
@@ -158,12 +158,10 @@ QuicVersionVector QuicSupportedVersions() {
QuicTag QuicVersionToQuicTag(const QuicVersion version) {
switch (version) {
- case QUIC_VERSION_21:
- return MakeQuicTag('Q', '0', '2', '1');
- case QUIC_VERSION_22:
- return MakeQuicTag('Q', '0', '2', '2');
case QUIC_VERSION_23:
return MakeQuicTag('Q', '0', '2', '3');
+ case QUIC_VERSION_24:
+ return MakeQuicTag('Q', '0', '2', '4');
default:
// This shold be an ERROR because we should never attempt to convert an
// invalid QuicVersion to be written to the wire.
@@ -190,9 +188,8 @@ return #x
string QuicVersionToString(const QuicVersion version) {
switch (version) {
- RETURN_STRING_LITERAL(QUIC_VERSION_21);
- RETURN_STRING_LITERAL(QUIC_VERSION_22);
RETURN_STRING_LITERAL(QUIC_VERSION_23);
+ RETURN_STRING_LITERAL(QUIC_VERSION_24);
default:
return "QUIC_VERSION_UNSUPPORTED";
}
@@ -260,14 +257,6 @@ QuicAckFrame::QuicAckFrame()
QuicAckFrame::~QuicAckFrame() {}
-CongestionFeedbackMessageTCP::CongestionFeedbackMessageTCP()
- : receive_window(0) {
-}
-
-QuicCongestionFeedbackFrame::QuicCongestionFeedbackFrame() : type(kTCP) {}
-
-QuicCongestionFeedbackFrame::~QuicCongestionFeedbackFrame() {}
-
QuicRstStreamErrorCode AdjustErrorForVersion(
QuicRstStreamErrorCode error_code,
QuicVersion version) {
@@ -309,11 +298,6 @@ QuicFrame::QuicFrame(QuicAckFrame* frame)
ack_frame(frame) {
}
-QuicFrame::QuicFrame(QuicCongestionFeedbackFrame* frame)
- : type(CONGESTION_FEEDBACK_FRAME),
- congestion_feedback_frame(frame) {
-}
-
QuicFrame::QuicFrame(QuicStopWaitingFrame* frame)
: type(STOP_WAITING_FRAME),
stop_waiting_frame(frame) {
@@ -418,11 +402,6 @@ ostream& operator<<(ostream& os, const QuicFrame& frame) {
os << "type { ACK_FRAME } " << *(frame.ack_frame);
break;
}
- case CONGESTION_FEEDBACK_FRAME: {
- os << "type { CONGESTION_FEEDBACK_FRAME } "
- << *(frame.congestion_feedback_frame);
- break;
- }
case STOP_WAITING_FRAME: {
os << "type { STOP_WAITING_FRAME } " << *(frame.stop_waiting_frame);
break;
@@ -482,19 +461,6 @@ ostream& operator<<(ostream& os, const QuicStreamFrame& stream_frame) {
return os;
}
-ostream& operator<<(ostream& os,
- const QuicCongestionFeedbackFrame& congestion_frame) {
- os << "type: " << congestion_frame.type;
- switch (congestion_frame.type) {
- case kTCP: {
- const CongestionFeedbackMessageTCP& tcp = congestion_frame.tcp;
- os << " receive_window: " << tcp.receive_window;
- break;
- }
- }
- return os;
-}
-
QuicGoAwayFrame::QuicGoAwayFrame()
: error_code(QUIC_NO_ERROR),
last_good_stream_id(0) {
@@ -609,9 +575,6 @@ RetransmittableFrames::~RetransmittableFrames() {
case ACK_FRAME:
delete it->ack_frame;
break;
- case CONGESTION_FEEDBACK_FRAME:
- delete it->congestion_feedback_frame;
- break;
case STOP_WAITING_FRAME:
delete it->stop_waiting_frame;
break;

Powered by Google App Engine
This is Rietveld 408576698