Roll src/net/third_party/quiche/src/ a89eee387..7e8ae0dee (12 commits)

https://quiche.googlesource.com/quiche.git/+log/a89eee387569..7e8ae0dee24d

$ git log a89eee387..7e8ae0dee --date=short --no-merges --format='%ad %ae %s'
2019-11-26 renjietang Remove redundant private keyword.
2019-11-26 wub gfe-relnote: (n/a) In QuicTimeAccumulator, change a "static constexpr data member" to a "static constexpr function". No behavior change, not protected.
2019-11-26 renjietang Remove QuicSession::SendRstStreamInner() because the |close_write_side_only| code paths share so little with each other.
2019-11-26 bnc Do not call MarkConsumed() from OnHeadersFramePayload() after decoding error.
2019-11-26 quiche-dev Adds a unit test demonstrating that SpdyHeaderBlock can store header names containing uppercase characters.
2019-11-26 wub gfe-relnote: (n/a) Change member initializer list to default member initializers in QuicConnectionStats. No behavior change, not protected.
2019-11-25 wub Internal QUICHE change
2019-11-25 wub gfe-relnote: (n/a) Deprecate --gfe2_reloadable_flag_quic_bbr2_fix_inflight_bounds.
2019-11-25 wub gfe-relnote: For server side QUIC connection, mark version_negotiated_ earlier. Protected by --gfe2_reloadable_flag_quic_version_negotiated_by_default_at_server.
2019-11-25 wub gfe-relnote: (n/a) Refactor Bbr2ProbeBwMode::OnCongestionEvent such that it only has one return point. No behavior change, not protected.
2019-11-25 wub gfe-relnote: (n/a) Do not call Initialize() in QuicTransportSimpleServerSession's constructor. Not used in production, not protected.
2019-11-24 wub gfe-relnote: Refactor how QuicConnectionStats.slowstart_duration is calculated in QUIC BBR1, and start populate it in QUIC BBR2. Not protected. (Worst case we get some incorrect values in transport connection stats)

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

R=rch@chromium.org

Change-Id: I35ddd5af3c98af640a888bc8792d411c0a6c8ff0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1937656
Commit-Queue: Bin Wu <wub@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#719479}

DEPS

@@ -307,7 +307,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': 'a89eee38756967553cd5b406ba7a1a14e8df9de5',
+  'quiche_revision': '7e8ae0dee24ddec61496d4328691a2c4f5a91415',
   # 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/BUILD.gn

@@ -1513,6 +1513,7 @@ component("net") {
       "third_party/quiche/src/quic/core/quic_tag.h",
       "third_party/quiche/src/quic/core/quic_time.cc",
       "third_party/quiche/src/quic/core/quic_time.h",
+      "third_party/quiche/src/quic/core/quic_time_accumulator.h",
       "third_party/quiche/src/quic/core/quic_transmission_info.cc",
       "third_party/quiche/src/quic/core/quic_transmission_info.h",
       "third_party/quiche/src/quic/core/quic_types.cc",
@@ -5536,6 +5537,7 @@ test("net_unittests") {
     "third_party/quiche/src/quic/core/quic_stream_test.cc",
     "third_party/quiche/src/quic/core/quic_sustained_bandwidth_recorder_test.cc",
     "third_party/quiche/src/quic/core/quic_tag_test.cc",
+    "third_party/quiche/src/quic/core/quic_time_accumulator_test.cc",
     "third_party/quiche/src/quic/core/quic_time_test.cc",
     "third_party/quiche/src/quic/core/quic_time_wait_list_manager_test.cc",
     "third_party/quiche/src/quic/core/quic_trace_visitor_test.cc",

net/quic/quic_flags_list.h

@@ -267,13 +267,6 @@ QUIC_FLAG(bool,
 // If true, enable HTTP/2 default scheduling(round robin).
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_enable_rr_write_scheduler, true)
 
-// If true, QuicFramer does not create an encrypter/decrypter for the
-// ENCRYPTION_INITIAL level.
-QUIC_FLAG(
-    bool,
-    FLAGS_quic_reloadable_flag_quic_framer_doesnt_create_initial_encrypter,
-    true)
-
 // If true, treat queued QUIC packets as sent.
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_treat_queued_packets_as_sent,
@@ -282,7 +275,7 @@ QUIC_FLAG(bool,
 // Call NeuterHandshakePackets() at most once per connection.
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_neuter_handshake_packets_once2,
-          false)
+          true)
 
 // If true, support HTTP/3 priority in v99.
 QUIC_FLAG(bool, FLAGS_quic_allow_http3_priority, false)
@@ -337,18 +330,18 @@ QUIC_FLAG(int32_t, FLAGS_quic_max_congestion_window, 2000)
 // encryption level.
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_use_connection_encryption_level,
-          false)
+          true)
 
 // If true, do not inject bandwidth in BbrSender::AdjustNetworkParameters.
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_bbr_donot_inject_bandwidth,
-          false)
+          true)
 
 // If true, close connection if CreateAndSerializeStreamFrame fails.
 QUIC_FLAG(
     bool,
     FLAGS_quic_reloadable_flag_quic_close_connection_on_failed_consume_data_fast_path,
-    false)
+    true)
 
 // If true, add a up call when N packet numbers get skipped.
 QUIC_FLAG(bool,
@@ -376,13 +369,27 @@ QUIC_FLAG(int32_t, FLAGS_quic_bbr2_default_startup_full_loss_count, 8)
 // upon high loss.
 QUIC_FLAG(double, FLAGS_quic_bbr2_default_inflight_hi_headroom, 0.01)
 
-// If true, for QUIC BBRv2: 1) don't grow inflight_hi unless it's fully used,
-// and 2) cap inflight_lo in PROBE_CRUISE.
-QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_bbr2_fix_inflight_bounds, true)
-
 // If true, when a stream receives data with wrong close offset, it closes the
 // connection. And the stream frame data will be discarded.
 QUIC_FLAG(
     bool,
     FLAGS_quic_reloadable_flag_quic_close_connection_and_discard_data_on_wrong_offset,
     false)
+
+// If true, log number of ack aggregation epochs in QUIC transport connection
+// stats.
+QUIC_FLAG(bool,
+          FLAGS_quic_reloadable_flag_quic_log_ack_aggregation_stats,
+          false)
+
+// If true, for server QUIC connections, set version_negotiated_ to true by
+// default.
+QUIC_FLAG(
+    bool,
+    FLAGS_quic_reloadable_flag_quic_version_negotiated_by_default_at_server,
+    false)
+
+// If true, QuicSession::SendRstStreamInner will be factored out and deleted.
+QUIC_FLAG(bool,
+          FLAGS_quic_reloadable_flag_quic_delete_send_rst_stream_inner,
+          false)