)]}'
{
  "commit": "afc9c139de7ea3a911171da83ac6922a0089fe34",
  "tree": "d76c466d4f4cd5da5cbc2ef42fd89b454bcc6134",
  "parents": [
    "ad54e957a81258f0bec879e1ff6fde0272f70099"
  ],
  "author": {
    "name": "Jiawei Shao",
    "email": "jiawei.shao@intel.com",
    "time": "Sun Dec 15 20:53:11 2024 -0800"
  },
  "committer": {
    "name": "Dawn LUCI CQ",
    "email": "dawn-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Sun Dec 15 20:53:11 2024 -0800"
  },
  "message": "D3D12: Hard-enable `D3D12SplitBufferTextureCopyForRowsPerImagePaddings`\n\nThe toggle `D3D12SplitBufferTextureCopyForRowsPerImagePaddings` doesn\u0027t\nneed to be defined as the code path it controlled before should always\nbe executed. It is because:\n1. In D3D12, the concept of `bufferLocation` specifies how to treat a\n   region of a buffer as a texture. In buffer-texture copies\n   `rowsPerImage` is used as the height of the `bufferLocation`, so if\n   the buffer is not large enough to hold a complete `rowsPerImage`\n   D3D12 debug layer will generate an error that the buffer is not large\n   enough to hold a complete `bufferLocation`, which fully meets the\n   concept in D3D12 and cannot say it a `bug` of D3D12 runtime.\n2. Although `UnrestrictedBufferTextureCopyPitchSupported` has been used\n   in Dawn we still need to split a buffer-texture copy into two in\n   some certain situations because of (1).\n\nBug: 42240278\nChange-Id: I83981d92057b79661aa39ede66a611329b7cf8a4\nReviewed-on: https://dawn-review.googlesource.com/c/dawn/+/218915\nCommit-Queue: Jiawei Shao \u003cjiawei.shao@intel.com\u003e\nReviewed-by: Kai Ninomiya \u003ckainino@chromium.org\u003e\nReviewed-by: Loko Kung \u003clokokung@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "c5d1cbac360f6a7fae3f2be78785fdc3aa184849",
      "old_mode": 33188,
      "old_path": "src/dawn/native/Toggles.cpp",
      "new_id": "8248c1771274cb4467140273c78448912f8b6f51",
      "new_mode": 33188,
      "new_path": "src/dawn/native/Toggles.cpp"
    },
    {
      "type": "modify",
      "old_id": "f096d3559d802a013affd472d123f8025aa02ded",
      "old_mode": 33188,
      "old_path": "src/dawn/native/Toggles.h",
      "new_id": "0993ef9f1ef656c23b6e0496ebaa5d32e291f377",
      "new_mode": 33188,
      "new_path": "src/dawn/native/Toggles.h"
    },
    {
      "type": "modify",
      "old_id": "10b6e0add179ce153160fed3b9c028fd3c3eb2d3",
      "old_mode": 33188,
      "old_path": "src/dawn/native/d3d12/PhysicalDeviceD3D12.cpp",
      "new_id": "58789aeeb489b4341c150c7e692a9b6c9c6b0612",
      "new_mode": 33188,
      "new_path": "src/dawn/native/d3d12/PhysicalDeviceD3D12.cpp"
    },
    {
      "type": "modify",
      "old_id": "f79591af419cbfe81cb000c7c193247e61f667d7",
      "old_mode": 33188,
      "old_path": "src/dawn/native/d3d12/UtilsD3D12.cpp",
      "new_id": "946aa262658ca6cc949037dc97f896c9d218e899",
      "new_mode": 33188,
      "new_path": "src/dawn/native/d3d12/UtilsD3D12.cpp"
    },
    {
      "type": "modify",
      "old_id": "1704a8920d205f202c9eeec5f0900343464188f4",
      "old_mode": 33188,
      "old_path": "src/dawn/tests/end2end/RequiredBufferSizeInCopyTests.cpp",
      "new_id": "35b9e182d1bbe194a5ebaefe1e9e07bb9a5720bf",
      "new_mode": 33188,
      "new_path": "src/dawn/tests/end2end/RequiredBufferSizeInCopyTests.cpp"
    }
  ]
}