)]}'
{
  "commit": "d48a92e533a0b7f45400f7f2686041161c23c8cd",
  "tree": "12bafaba1ae8ac0b9ecd2593ddab03a8644f1478",
  "parents": [
    "bf82b02e888d7acb087655103b7c86b7534e8249"
  ],
  "author": {
    "name": "bnc",
    "email": "bnc@google.com",
    "time": "Tue Jun 18 19:24:58 2019 -0700"
  },
  "committer": {
    "name": "Bence Béky",
    "email": "bnc@google.com",
    "time": "Wed Jun 19 14:24:13 2019 +0000"
  },
  "message": "Add DCHECK to v99 code path in QuicSpdyStream.\n\nWe really should not feed any data to HttpDecoder if sequencer is closed, that\nis, at least |close_offset_| bytes have been consumed.  See\nhttps://cs.corp.google.com/quiche/quic/core/quic_stream_sequencer.cc?l\u003d199.\n\nRemove QuicSpdyStreamTest.ReceivingTrailersOnRequestStream test which violates\nthis assumption because it feeds trailers into the stream before data.  If order\nis fixed, this test becomes redundant with\nQuicSpdyStreamTest.ProcessBodyAfterTrailers, which this CL improves a bit.\n\nThis came up during my investigation of https://crbug.com/969391, where the\ninput data found by the fuzzer would trip over this new DCHECK.\n\nPiperOrigin-RevId: 253916860\nChange-Id: I489710f132c7b6d1b00a203d871701bc2b52685c\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "6c1f2add01739032f2bf758e306da254bcb7d38d",
      "old_mode": 33188,
      "old_path": "quic/core/http/quic_spdy_stream.cc",
      "new_id": "aa20771125f71bb681d8b124f4187c23ce70e0d2",
      "new_mode": 33188,
      "new_path": "quic/core/http/quic_spdy_stream.cc"
    },
    {
      "type": "modify",
      "old_id": "64e45b999fb8f2a4b9972c5e463c16e86150c227",
      "old_mode": 33188,
      "old_path": "quic/core/http/quic_spdy_stream_test.cc",
      "new_id": "63d5ba3d96198ea97dcaa766628a50d9ff1e6f73",
      "new_mode": 33188,
      "new_path": "quic/core/http/quic_spdy_stream_test.cc"
    }
  ]
}