)]}'
{
  "commit": "def41cc7b1960b32e6ac8aa22293ee4be1cc2c4c",
  "tree": "0a85536f7ad927141a442a189939b7437a691e7e",
  "parents": [
    "f196d18c5b93509fc470527061a85d100ed4ba68"
  ],
  "author": {
    "name": "Nate Bosch",
    "email": "nbosch@google.com",
    "time": "Mon Oct 23 15:42:03 2023 -0700"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Mon Oct 23 15:42:03 2023 -0700"
  },
  "message": "Tweak dart2js source map rewriting (#2128)\n\n- Remove a TODO referencing a closed issue. The issue was deduped to\r\n  another which was closed without changes. Manual testing in a browser\r\n  shows that the rewriting is still necessary.\r\n- Extract a variable for the source map path as a Uri instead of\r\n  computing it within the loop.\r\n- Directly return URIs which are already `file://`. This would be a\r\n  slight behavior change for the `runInBrowser.dart` synthetic source if\r\n  it was ever put in the source map file with an absolute `file://`\r\n  URI, but with the current (and apparently planned) dart2js output they\r\n  do not have the file scheme. This is not a behavior change for any\r\n  other cases where the source is a `file` URI, since it is no necessary\r\n  to call `resolveUri` with an absolute path, and all `file` URI are\r\n  absolute.\r\n- Refactor for readability - use a list literal, `Map.update`, and a `switch`.\r\n\r\nA sample run with a `Stopwatch` shows this reduces the non-IO overhead\r\nby about 75% (out of 165ms in my sample with multiple suites).",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "343ce0c6b88ef9e1dfe78f27ef27fd3c583a1916",
      "old_mode": 33188,
      "old_path": "pkgs/test_core/lib/src/runner/dart2js_compiler_pool.dart",
      "new_id": "a13a65aed6570aa89ad329ff2de6fd1fe366314f",
      "new_mode": 33188,
      "new_path": "pkgs/test_core/lib/src/runner/dart2js_compiler_pool.dart"
    }
  ]
}