)]}'
{
  "commit": "b402517c01d42d3930b8add316ae6d74b9712cbd",
  "tree": "0ad50e35cd1bb97b63d15501adb6fbf70c4e6ea1",
  "parents": [
    "1ad8dc6e917d8df681cf341f8fbb938fa584dace"
  ],
  "author": {
    "name": "bnc",
    "email": "bnc@google.com",
    "time": "Fri Jul 12 18:09:59 2019 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Fri Jul 12 18:12:01 2019 -0700"
  },
  "message": "Use HpackEntry::Size() instead of rolling our own in QpackDynamicTable.\n\n(Note that QpackEntry is an alias for HpackEntry.)\n\nI\u0027m a little uncomfortable with HpackEntry::Size() returning size_t instead of\nuint64_t, because Chromium supports platforms where size_t is uint32_t and\nsometimes funny things happen, especially with UBSAN around type conversions.\nThe reason for returning size_t is that is string_view::size_type and\nstd::string::size_type.  In practice we\u0027ll never deal with long strings so\nthis should really not be a problem.\n\nThis came up because I will need to access EntrySize() from outside\nQpackHeaderTable and it makes more sense to use HpackEntry::Size() consistently\nthen to make EntrySize() public.\n\ngfe-relnote: n/a, QUIC v99-only code.\nPiperOrigin-RevId: 257903243\nChange-Id: I354ffb46a9a6d7e2e4f2ba2d581115f6c0f4cfd5\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "9522666dc7941d84cf272700d18004089da6a55e",
      "old_mode": 33188,
      "old_path": "quic/core/qpack/qpack_header_table.cc",
      "new_id": "6fa64125a1a8d893b7f2db65d5c2e0e08f33646c",
      "new_mode": 33188,
      "new_path": "quic/core/qpack/qpack_header_table.cc"
    }
  ]
}
