)]}'
{
  "commit": "8ea3af9118a65387fe4d8428db6f59c5927dd55a",
  "tree": "4a4ac6c29467a4872be18336c63c3ea67216dc67",
  "parents": [
    "efa0e81145f19ef551f7025d6940cc2dfd15af8c"
  ],
  "author": {
    "name": "Andrew Grieve",
    "email": "agrieve@google.com",
    "time": "Tue Jun 10 12:59:34 2025 -0700"
  },
  "committer": {
    "name": "GN LUCI",
    "email": "gn-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Tue Jun 10 13:23:09 2025 -0700"
  },
  "message": "Revert \"Run Target::OnResolve() on background threads\"\n\nThis reverts commit 3ef7da38dcfb63212d97a987d5c60374134e1007.\n\nReason for revert: Failing tests /w asan (and on release bots)\n\nBug: chromium:398002893\nOriginal change\u0027s description:\n\u003e Run Target::OnResolve() on background threads\n\u003e\n\u003e OnResolve() is often trivial, but it can also not be, as there are a lot\n\u003e of possible checks that it can do.\n\u003e\n\u003e \"gn --tracelog gen\" now shows all threads being always busy.\n\u003e\n\u003e Hyperfine Benchmark from Chromium:\n\u003e Before:\n\u003e   Time (mean ± σ):      7.349 s ±  0.251 s    [User: 66.903 s, System: 50.486 s]\n\u003e   Range (min … max):    6.974 s …  7.655 s    10 runs\n\u003e\n\u003e After:\n\u003e   Time (mean ± σ):      6.563 s ±  0.313 s    [User: 67.873 s, System: 53.023 s]\n\u003e   Range (min … max):    6.186 s …  7.281 s    10 runs\n\u003e Bug: chromium:398002893\n\u003e Change-Id: I30bd9b7541a961b1d0cef80c229b0a3f24752d3e\n\u003e Reviewed-on: https://gn-review.googlesource.com/c/gn/+/19020\n\u003e Reviewed-by: David Turner \u003cdigit@google.com\u003e\n\u003e Commit-Queue: David Turner \u003cdigit@google.com\u003e\n\u003e Reviewed-by: Takuto Ikuta \u003ctikuta@google.com\u003e\n\n# Not skipping CQ checks because original CL landed \u003e 1 day ago.\n\nBug: chromium:398002893\nChange-Id: I4ca499e210288b46a91209967a635004c9d0ad68\nReviewed-on: https://gn-review.googlesource.com/c/gn/+/19060\nReviewed-by: Andrew Grieve \u003cagrieve@google.com\u003e\nCommit-Queue: Roland McGrath \u003cmcgrathr@google.com\u003e\nReviewed-by: Roland McGrath \u003cmcgrathr@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "8718822dbb27a494c741de219c93662947627659",
      "old_mode": 33188,
      "old_path": "src/gn/analyzer_unittest.cc",
      "new_id": "dfd14865b946358b625dd8373a5f56e83adc9c55",
      "new_mode": 33188,
      "new_path": "src/gn/analyzer_unittest.cc"
    },
    {
      "type": "modify",
      "old_id": "bf47b13b21b0197bbae4e459481d5bc976b6f455",
      "old_mode": 33188,
      "old_path": "src/gn/builder.cc",
      "new_id": "1f3ecd8ff50643f85e0e1ab3ca446c1e94982dfe",
      "new_mode": 33188,
      "new_path": "src/gn/builder.cc"
    },
    {
      "type": "modify",
      "old_id": "e4d996c9ad2870ca4d1d712b65c9e743abf06513",
      "old_mode": 33188,
      "old_path": "src/gn/builder.h",
      "new_id": "a9c01aa491dec9cc5e395a36fbb65406e35e4a71",
      "new_mode": 33188,
      "new_path": "src/gn/builder.h"
    },
    {
      "type": "modify",
      "old_id": "59b58f8130ffddc1512667386950f2ae7141d536",
      "old_mode": 33188,
      "old_path": "src/gn/builder_unittest.cc",
      "new_id": "59068a6e8f14cc93d3167065eea8d1261418304a",
      "new_mode": 33188,
      "new_path": "src/gn/builder_unittest.cc"
    },
    {
      "type": "modify",
      "old_id": "4a6e193bfa8c1bf6061e36d0385d164fb7c35425",
      "old_mode": 33188,
      "old_path": "src/gn/item.h",
      "new_id": "dd1a60662fbf1376025c50946693155313c81541",
      "new_mode": 33188,
      "new_path": "src/gn/item.h"
    }
  ]
}
