)]}'
{
  "commit": "1820314b7860d1f73327cbd97cd0be3b35e4f618",
  "tree": "958f7aed60e163faa1bcec5a3ed229e7e64f194f",
  "parents": [
    "75c7ea51042b3846ccd0276a686f5bad440ea906"
  ],
  "author": {
    "name": "bnc",
    "email": "bnc@google.com",
    "time": "Fri Jun 11 12:09:24 2021 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Fri Jun 11 12:10:41 2021 -0700"
  },
  "message": "Add QuicCryptoClientConfigTest.MultipleCanonicalEntries.\n\nWhen working on cl/378863627 I originally wrote:\n\n  auto it \u003d canonical_server_map_.lower_bound(suffix_server_id);\n  if (it \u003d\u003d canonical_server_map_.end() || it-\u003efirst \u003c suffix_server_id) {\n    canonical_server_map_.insert(...);\n    return false;\n  }\n\nThis is incorrect: lower_bound() never returns an iterator with key less than\nits argument.  No tests caught this.  The statement should have been\n  if (it \u003d\u003d canonical_server_map_.end() || suffix_server_id \u003c it-\u003efirst) {\nor simply\n  if (it \u003d\u003d canonical_server_map_.end() || it-\u003efirst !\u003d suffix_server_id) {\n\nThis CL adds a test that would have failed with the incorrect statement.\n\nPiperOrigin-RevId: 378922152\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "6dd09677fb9a396279ce08ed2b5eccd0cfc7ec6b",
      "old_mode": 33188,
      "old_path": "quic/core/crypto/quic_crypto_client_config_test.cc",
      "new_id": "5bcf01cd07425c9c77e072ae3d3093e8d7cdd59d",
      "new_mode": 33188,
      "new_path": "quic/core/crypto/quic_crypto_client_config_test.cc"
    }
  ]
}