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

Side by Side Diff: net/quic/quic_config.cc

Issue 647933002: Call SetDefaults from the QuicConfig constructor instead of requiring (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Allow_number_of_undecryptable_packets_76707156
Patch Set: Created 6 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 unified diff | Download patch
« no previous file with comments | « net/quic/quic_config.h ('k') | net/quic/quic_config_test.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/quic/quic_config.h" 5 #include "net/quic/quic_config.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "net/quic/crypto/crypto_handshake_message.h" 10 #include "net/quic/crypto/crypto_handshake_message.h"
(...skipping 426 matching lines...) Expand 10 before | Expand all | Expand 10 after
437 initial_round_trip_time_us_(kIRTT, PRESENCE_OPTIONAL), 437 initial_round_trip_time_us_(kIRTT, PRESENCE_OPTIONAL),
438 // TODO(rjshade): Remove this when retiring QUIC_VERSION_19. 438 // TODO(rjshade): Remove this when retiring QUIC_VERSION_19.
439 initial_flow_control_window_bytes_(kIFCW, PRESENCE_OPTIONAL), 439 initial_flow_control_window_bytes_(kIFCW, PRESENCE_OPTIONAL),
440 // TODO(rjshade): Make this PRESENCE_REQUIRED when retiring 440 // TODO(rjshade): Make this PRESENCE_REQUIRED when retiring
441 // QUIC_VERSION_19. 441 // QUIC_VERSION_19.
442 initial_stream_flow_control_window_bytes_(kSFCW, PRESENCE_OPTIONAL), 442 initial_stream_flow_control_window_bytes_(kSFCW, PRESENCE_OPTIONAL),
443 // TODO(rjshade): Make this PRESENCE_REQUIRED when retiring 443 // TODO(rjshade): Make this PRESENCE_REQUIRED when retiring
444 // QUIC_VERSION_19. 444 // QUIC_VERSION_19.
445 initial_session_flow_control_window_bytes_(kCFCW, PRESENCE_OPTIONAL), 445 initial_session_flow_control_window_bytes_(kCFCW, PRESENCE_OPTIONAL),
446 socket_receive_buffer_(kSRBF, PRESENCE_OPTIONAL) { 446 socket_receive_buffer_(kSRBF, PRESENCE_OPTIONAL) {
447 SetDefaults();
447 } 448 }
448 449
449 QuicConfig::~QuicConfig() {} 450 QuicConfig::~QuicConfig() {}
450 451
451 void QuicConfig::SetCongestionFeedback( 452 void QuicConfig::SetCongestionFeedback(
452 const QuicTagVector& congestion_feedback, 453 const QuicTagVector& congestion_feedback,
453 QuicTag default_congestion_feedback) { 454 QuicTag default_congestion_feedback) {
454 congestion_feedback_.set(congestion_feedback, default_congestion_feedback); 455 congestion_feedback_.set(congestion_feedback, default_congestion_feedback);
455 } 456 }
456 457
(...skipping 256 matching lines...) Expand 10 before | Expand all | Expand 10 after
713 peer_hello, hello_type, error_details); 714 peer_hello, hello_type, error_details);
714 } 715 }
715 if (error == QUIC_NO_ERROR) { 716 if (error == QUIC_NO_ERROR) {
716 error = connection_options_.ProcessPeerHello( 717 error = connection_options_.ProcessPeerHello(
717 peer_hello, hello_type, error_details); 718 peer_hello, hello_type, error_details);
718 } 719 }
719 return error; 720 return error;
720 } 721 }
721 722
722 } // namespace net 723 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/quic_config.h ('k') | net/quic/quic_config_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698