Roll src/net/third_party/quiche/src/ 4cc508c72..b38e63550 (12 commits)

https://quiche.googlesource.com/quiche.git/+log/4cc508c720ac..3ccd7703dc37

$ git log 4cc508c72..b38e63550 --date=short --no-merges --format='%ad %ae %s'
2019-10-24 renjietang Make QuicStreamTest to check on OnDataAvailable() calls.
2019-10-24 wub Fix 10 ClangTidy - Performance findings in //depot/google3/third_party/quic/.
2019-10-24 wub Fix 2 clang-tidy findings in third_party/quic.
2019-10-24 fayang Remove unused variables in QuicVersionsTest.QuicEnableVersion.
2019-10-24 ianswett Add flag counts for two flags that had none.
2019-10-24 dschinazi Fix QUIC_EXPORT_PRIVATE compilation on Windows
2019-10-23 nharper gfe-relnote: deprecate gfe2_reloadable_flag_quic_enable_version_48_2
2019-10-23 renjietang Close connection when stream receives wrong data regarding close offset.
2019-10-23 vasilvv Add a simple integration test for QuicTransport.
2019-10-23 dschinazi Enable -Wexit-time-destructors in QUIC
2019-10-23 dschinazi Add QUIC_EXPORT_PRIVATE to prepare for presubmit
2019-10-23 fayang gfe-relnote: Add QuicCoalescedPacket class. Not used in prod, not protected.

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

Change-Id: Idd70e621540c3f4d52dcd9854a243d1a34dd0d41
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1878628Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#709199}

DEPS

@@ -314,7 +314,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': '4cc508c720aca5e15d1e5d89b970f58c57414764',
+  'quiche_revision': 'b38e635502035083ead0d64e26026ee724238c65',
   # 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.

components/domain_reliability/quic_error_mapping.cc

@@ -316,6 +316,10 @@ const struct QuicErrorMapping {
     {quic::QUIC_QPACK_ENCODER_STREAM_ERROR, "QUIC.QPACK.ENCODER.STREAM.ERROR"},
     {quic::QUIC_QPACK_DECODER_STREAM_ERROR, "QUIC.QPACK.DECODER.STREAM.ERROR"},
 
+    {quic::QUIC_STREAM_DATA_BEYOND_CLOSE_OFFSET,
+     "quic.stream.data.beyond.close.offset"},
+    {quic::QUIC_STREAM_MULTIPLE_OFFSET, "quic.stream.multiple.offset"},
+
     // QUIC_INVALID_APPLICATION_CLOSE_DATA was code 101. The code has been
     // deprecated, but to keep the assert below happy, there needs to be
     // an entry for it, but the symbol is gone.

net/BUILD.gn

@@ -1575,6 +1575,8 @@ component("net") {
       "third_party/quiche/src/quic/core/quic_buffer_allocator.h",
       "third_party/quiche/src/quic/core/quic_buffered_packet_store.cc",
       "third_party/quiche/src/quic/core/quic_buffered_packet_store.h",
+      "third_party/quiche/src/quic/core/quic_coalesced_packet.cc",
+      "third_party/quiche/src/quic/core/quic_coalesced_packet.h",
       "third_party/quiche/src/quic/core/quic_config.cc",
       "third_party/quiche/src/quic/core/quic_config.h",
       "third_party/quiche/src/quic/core/quic_connection.cc",
@@ -5589,6 +5591,7 @@ test("net_unittests") {
     "third_party/quiche/src/quic/core/quic_arena_scoped_ptr_test.cc",
     "third_party/quiche/src/quic/core/quic_bandwidth_test.cc",
     "third_party/quiche/src/quic/core/quic_buffered_packet_store_test.cc",
+    "third_party/quiche/src/quic/core/quic_coalesced_packet_test.cc",
     "third_party/quiche/src/quic/core/quic_config_test.cc",
     "third_party/quiche/src/quic/core/quic_connection_id_test.cc",
     "third_party/quiche/src/quic/core/quic_connection_test.cc",

net/quic/quic_flags_list.h

@@ -159,9 +159,6 @@ QUIC_FLAG(bool,
 // If true, enable QUIC version 47.
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_enable_version_47, false)
 
-// If true, enable QUIC version 48.
-QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_enable_version_48_2, true)
-
 // If true, disable QUIC version 39.
 QUIC_FLAG(bool, FLAGS_quic_reloadable_flag_quic_disable_version_39, true)
 
@@ -411,3 +408,9 @@ QUIC_FLAG(bool,
 QUIC_FLAG(bool,
           FLAGS_quic_reloadable_flag_quic_no_decrease_in_final_offset,
           false)
+
+// If true, connection will be closed if a stream receives stream frame or
+// RESET_STREAM frame with bad close offset.
+QUIC_FLAG(bool,
+          FLAGS_quic_reloadable_flag_quic_close_connection_on_wrong_offset,
+          false)