)]}'
{
  "commit": "eb636a3bea8bfac1abcaac4c8465be14865da073",
  "tree": "c72f5219d2337063f1bfa40469a4f89606edb20a",
  "parents": [
    "9fa4c12a40fe32c193c16acf0ad5c4bf1056aa36"
  ],
  "author": {
    "name": "Lei Zhang",
    "email": "thestig@chromium.org",
    "time": "Fri Mar 28 22:50:29 2025 -0700"
  },
  "committer": {
    "name": "Pdfium LUCI CQ",
    "email": "pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Fri Mar 28 22:50:29 2025 -0700"
  },
  "message": "Refine NeedToPremultiplyBitmap() logic\n\nCFX_DIBitmap::ScopedPremultiplier::NeedToPremultiplyBitmap() determines\nif a bitmap needs to be pre-multiplied, yet it does not check the bitmap\nat all. This is because it is relying on two crutches:\n\n1) CFX_DIBitmap::PreMultiply() and CFX_DIBitmap::UnPreMultiply() are\n   no-ops for formats without alpha.\n2) It never handles pre-multiplied bitmaps.\n\nCondition (2) will go away in the future, so anticipate that and change\nNeedToPremultiplyBitmap() to check the bitmap\u0027s format.\n\nBug: 42271033\nChange-Id: I4d961c5fa20bd254962562e6ce51ef4236ef7360\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/129012\nReviewed-by: Tom Sepez \u003ctsepez@chromium.org\u003e\nCommit-Queue: Lei Zhang \u003cthestig@chromium.org\u003e\nReviewed-by: Thomas Sepez \u003ctsepez@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "6b9b1ca88f548beaa435039ea1172478501ce070",
      "old_mode": 33188,
      "old_path": "core/fxge/dib/cfx_dibitmap.cpp",
      "new_id": "de5a5c4430cef80f4d31157ebe0912f900b429c3",
      "new_mode": 33188,
      "new_path": "core/fxge/dib/cfx_dibitmap.cpp"
    },
    {
      "type": "modify",
      "old_id": "b0979361a494ba50df1655fe4510e2fa2de03a19",
      "old_mode": 33188,
      "old_path": "core/fxge/dib/cfx_dibitmap.h",
      "new_id": "c5ef3aa88bb7e7cfe48f9a25500ce483045ccab1",
      "new_mode": 33188,
      "new_path": "core/fxge/dib/cfx_dibitmap.h"
    }
  ]
}
