)]}'
{
  "commit": "bcf3c48e3f5cd725daa84655dd5249f5e422ed59",
  "tree": "830a16dd30c6182fd235a5ef95770538b4ec1ec6",
  "parents": [
    "e782e37b182c767925a750dfad5aa8fe979936b4"
  ],
  "author": {
    "name": "Hui Yingst",
    "email": "nigi@chromium.org",
    "time": "Tue May 12 00:22:33 2020 +0000"
  },
  "committer": {
    "name": "Chromium commit bot",
    "email": "commit-bot@chromium.org",
    "time": "Tue May 12 00:22:33 2020 +0000"
  },
  "message": "Check whether the object is a page object before updating |m_PageList|.\n\nbug_1491.in is rendered blank because when handling the \"GoTo\" action,\nthe destination points to an object which is a /Pages object instead\nof a /Page object. This sets |m_PageList[0]| with the wrong object\nwhich is missing contents.\n\nThis CL adds a check for valid page object before updating |m_PageList|\nin CPDF_Document::GetPageIndex(), so that when processing the \"GoTo\"\naction, |m_PageList[i]| will remain unchanged for page |i| if the\nmatching object number doesn\u0027t point to a /Page object. Later, during\nthe rendering process, CPDF_Document::GetPageDictionary() can rely on\nTraversePDFPages() to update |m_PageList| with more reliable object\nnumbers.\n\nBug: pdfium:1491\nChange-Id: I4e8a81c6504f52dd5e5b8cd9195c2c60d8885f94\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/69710\nCommit-Queue: Hui Yingst \u003cnigi@chromium.org\u003e\nReviewed-by: Lei Zhang \u003cthestig@chromium.org\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "76c65b58d679f899ff75978854e0b2c5a2238860",
      "old_mode": 33188,
      "old_path": "core/fpdfapi/parser/cpdf_document.cpp",
      "new_id": "e4f246ccea5d058b8d9444d73f97f270b798e8e8",
      "new_mode": 33188,
      "new_path": "core/fpdfapi/parser/cpdf_document.cpp"
    },
    {
      "type": "modify",
      "old_id": "0748e66dd072ed87170daf3d472675ba48aec43a",
      "old_mode": 33188,
      "old_path": "testing/SUPPRESSIONS",
      "new_id": "da51760dc3d6b4ca351c6a13ca8ae87c94de1170",
      "new_mode": 33188,
      "new_path": "testing/SUPPRESSIONS"
    }
  ]
}