Roll src/net/third_party/quiche/src/ d2bf0ba74..e1e5e502e (7 commits)

https://quiche.googlesource.com/quiche.git/+log/d2bf0ba74c74..e1e5e502ea64

$ git log d2bf0ba74..e1e5e502e --date=short --no-merges --format='%ad %ae %s'
2020-08-17 wub Move the implementation of GetReleaseTime from QuicBatchWriterBase to QuicGsoBatchWriter. No behavior change.
2020-08-17 dschinazi Deprecate quic_google_transport_param_omit_old
2020-08-17 renjietang Move OnZeroRttRejected() debug event from HTTP layer to QUIC transport layer.
2020-08-17 fayang Deprecate gfe2_reloadable_flag_quic_support_handshake_done_in_t050.
2020-08-14 wub Internal QUICHE change
2020-08-14 wub Default enable MTU discovery in QUIC server. Protected by --gfe2_reloadable_flag_quic_enable_mtu_discovery_at_server.
2020-08-14 fayang Stop blackhole detection when a duplicate ACK causes packets to be detected lost. Protected by existing gfe2_reloadable_flag_quic_default_enable_5rto_blackhole_detection2.

Created with:
  roll-dep src/net/third_party/quiche/src src/third_party/quic_trace/src

Change-Id: Ic442251f8328e5bbfaaf84b1bf9aea211c28e413
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360926
Auto-Submit: Haoyue Wang <haoyuewang@chromium.org>
Commit-Queue: David Schinazi <dschinazi@chromium.org>
Reviewed-by: David Schinazi <dschinazi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#799233}

DEPS

@@ -322,7 +322,7 @@ vars = {
   # Three lines of non-changing comments so that
   # the commit queue can handle CLs rolling feed
   # and whatever else without interference from each other.
-  'quiche_revision': 'd2bf0ba74c742056b9f982d35fccff77cabb4c59',
+  'quiche_revision': 'e1e5e502ea646ae86f933cf616cfff95248b7fef',
   # Three lines of non-changing comments so that
   # the commit queue can handle CLs rolling ios_webkit
   # and whatever else without interference from each other.

net/quic/mock_crypto_client_stream.cc

@@ -342,7 +342,7 @@ void MockCryptoClientStream::SetConfigNegotiated() {
     TransportParameters params;
     ASSERT_TRUE(config.FillTransportParameters(&params));
     error = session()->config()->ProcessTransportParameters(
-        params, CLIENT, /*is_resumption=*/false, &error_details);
+        params, /*is_resumption=*/false, &error_details);
   } else {
     CryptoHandshakeMessage msg;
     config.ToHandshakeMessage(

net/quic/quic_flags_list.h

@@ -260,7 +260,7 @@ QUIC_FLAG(bool,
           false)
 
 // If true, enables support for TLS resumption in QUIC.
-QUIC_FLAG(bool, FLAGS_quic_restart_flag_quic_enable_tls_resumption_v4, false)
+QUIC_FLAG(bool, FLAGS_quic_restart_flag_quic_enable_tls_resumption_v4, true)
 
 // When true, QUIC's BBRv2 ignores inflight_lo in PROBE_BW.
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_bbr2_ignore_inflight_lo, true)
@@ -271,12 +271,6 @@ QUIC_FLAG(bool, FLAGS_quic_restart_flag_quic_enable_zero_rtt_for_tls_v2, true)
 // If true, default on PTO which unifies TLP + RTO loss recovery.
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_default_on_pto, false)
 
-// When true, QUIC+TLS will not send nor parse the old-format Google-specific
-// transport parameters.
-QUIC_FLAG(bool,
-          FLAGS_quic_restart_flag_quic_google_transport_param_omit_old,
-          true)
-
 // If true, the B2HI connection option limits reduction of inflight_hi to
 // (1-Beta)*CWND.
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_bbr2_limit_inflight_hi, false)
@@ -293,11 +287,6 @@ QUIC_FLAG(
 // If true, disable QUIC version h3-29.
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_disable_version_draft_29, false)
 
-// If true, support HANDSHAKE_DONE frame in T050
-QUIC_FLAG(bool,
-          FLAGS_quic_reloadable_flag_quic_support_handshake_done_in_t050,
-          true)
-
 // When true, QuicDispatcher supports decapsulation of Legacy Version
 // Encapsulation packets.
 QUIC_FLAG(
@@ -347,7 +336,7 @@ QUIC_FLAG(
 // mitigate RTT inflations.
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_coalesced_packet_of_higher_space2,
-          false)
+          true)
 
 // If true, record the received min_ack_delay in transport parameters to QUIC
 // config.
@@ -395,9 +384,7 @@ QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_http3_goaway_new_behavior, true)
 
 // If true, QUIC connection will revert to a previously validated MTU (if
 // exists) after two PTOs.
-QUIC_FLAG(bool,
-          FLAGS_quic_reloadable_flag_quic_revert_mtu_after_two_ptos,
-          false)
+QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_revert_mtu_after_two_ptos, true)
 
 // Simplify the ACK code in quic_received_packet_manager.
 QUIC_FLAG(bool,
@@ -411,16 +398,14 @@ QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_use_half_rtt_as_first_pto, true)
 // supplied.
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_enable_overshooting_detection,
-          false)
+          true)
 
 // If true, enable QUIC version h3-T051.
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_enable_version_t051, true)
 
 // If true, fix a case where data is marked lost in HANDSHAKE level but
 // HANDSHAKE key gets decrypted later.
-QUIC_FLAG(bool,
-          FLAGS_quic_reloadable_flag_quic_fix_neuter_handshake_data,
-          false)
+QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_fix_neuter_handshake_data, true)
 
 // If true, when data is sending in fast path mode in the creator, making sure
 // stream data is sent in the right encryption level.
@@ -456,3 +441,8 @@ QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_fix_on_ping_timeout, true)
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_send_key_update_not_yet_supported,
           true)
+
+// If true, QUIC will default enable MTU discovery, with a target of 1450 bytes.
+QUIC_FLAG(bool,
+          FLAGS_quic_reloadable_flag_quic_enable_mtu_discovery_at_server,
+          false)