)]}'
{
  "commit": "c891fe06a87f52bca0f8b6d8f2e61460d0a57f32",
  "tree": "fd28528d3440aaa1023162e4e67c7b13d9d5d4f3",
  "parents": [
    "9b8f9c810a236df2a1ac48ab99c5964fb30f6754"
  ],
  "author": {
    "name": "rouslan",
    "email": "rouslan@chromium.org",
    "time": "Fri Feb 05 11:49:40 2016 -0800"
  },
  "committer": {
    "name": "Commit bot",
    "email": "commit-bot@chromium.org",
    "time": "Fri Feb 05 19:50:55 2016 +0000"
  },
  "message": "Revert of [GN] Don\u0027t rewrite files with the same contents (patchset #5 id:80001 of https://codereview.chromium.org/1656253003/ )\n\nReason for revert:\nNeed to revert this patch according https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues. This patch added a flaky test FilesystemUtils.WriteFileIfChanged.\n\n-----\n\nIt\u0027s a test flake:\n1) Try to find the patch that caused the flake. It should be recent (e.g. last day or two) in all likelihood.\n2) If successful with finding that patch, revert the patch. This is especially true if the flake is from a new test introduced in that patch.\n3) Close the bug.\n\n-----\n\nThe test has failed in the following builds:\n\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176911\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176911\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176735\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176715\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176715\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176561\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176462\nhttp://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/176375\n\n-----\n\nExample failure:\n\n[ RUN      ] FilesystemUtils.WriteFileIfChanged\n../../tools/gn/filesystem_utils_unittest.cc:610: Failure\nExpected: (last_modified) !\u003d (file_info.last_modified), actual: 2016-02-04 18:06:36.920 UTC vs 2016-02-04 18:06:36.920 UTC\n[  FAILED  ] FilesystemUtils.WriteFileIfChanged (3 ms)\n[315/315] FilesystemUtils.WriteFileIfChanged (3 ms)\nRetrying 1 test (retry #2)\n[ RUN      ] FilesystemUtils.WriteFileIfChanged\n../../tools/gn/filesystem_utils_unittest.cc:610: Failure\nExpected: (last_modified) !\u003d (file_info.last_modified), actual: 2016-02-04 18:06:36.936 UTC vs 2016-02-04 18:06:36.936 UTC\n[  FAILED  ] FilesystemUtils.WriteFileIfChanged (2 ms)\n[316/316] FilesystemUtils.WriteFileIfChanged (2 ms)\nRetrying 1 test (retry #3)\n[ RUN      ] FilesystemUtils.WriteFileIfChanged\n../../tools/gn/filesystem_utils_unittest.cc:610: Failure\nExpected: (last_modified) !\u003d (file_info.last_modified), actual: 2016-02-04 18:06:36.952 UTC vs 2016-02-04 18:06:36.952 UTC\n[  FAILED  ] FilesystemUtils.WriteFileIfChanged (2 ms)\n[317/317] FilesystemUtils.WriteFileIfChanged (2 ms)\n1 test failed:\n    FilesystemUtils.WriteFileIfChanged (../../tools/gn/filesystem_utils_unittest.cc:579)\n\n-----\n\nMore details in http://crbug.com/584548.\n\nOriginal issue\u0027s description:\n\u003e [GN] Don\u0027t rewrite files with the same contents\n\u003e\n\u003e BUG\u003d\n\u003e\n\u003e Committed: https://crrev.com/f8ea5cceefcedd4a01935d5ac4d2ba71e23ac13e\n\u003e Cr-Commit-Position: refs/heads/master@{#373544}\n\nTBR\u003dbrettw@chromium.org,tmoniuszko@opera.com\n# Not skipping CQ checks because original CL landed more than 1 days ago.\nBUG\u003d\n\nReview URL: https://codereview.chromium.org/1672783002\n\nCr-Original-Commit-Position: refs/heads/master@{#373882}\nCr-Mirrored-From: https://chromium.googlesource.com/chromium/src\nCr-Mirrored-Commit: ef389f44f3d90628cc61a294bf3e340dbccc1cd1\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "2ceca81677dbc68d0e041271ea9702ec1c3302c0",
      "old_mode": 33188,
      "old_path": "tools/gn/filesystem_utils.cc",
      "new_id": "5905cdb9f1ec73749a03e15428d5109542712b22",
      "new_mode": 33188,
      "new_path": "tools/gn/filesystem_utils.cc"
    },
    {
      "type": "modify",
      "old_id": "aaa08babde9e306ff82a0aca6b459ff30e170ee8",
      "old_mode": 33188,
      "old_path": "tools/gn/filesystem_utils.h",
      "new_id": "f64c00d751918238d685d49ef0701beab88bf62e",
      "new_mode": 33188,
      "new_path": "tools/gn/filesystem_utils.h"
    },
    {
      "type": "modify",
      "old_id": "e44d33797e8b5d7c5103a68439cea933b0c8fa5c",
      "old_mode": 33188,
      "old_path": "tools/gn/filesystem_utils_unittest.cc",
      "new_id": "27ccab2ea4d0db453c217d1921db71729de58aac",
      "new_mode": 33188,
      "new_path": "tools/gn/filesystem_utils_unittest.cc"
    },
    {
      "type": "modify",
      "old_id": "feb1004c3f22d2ef668294e1d1c8d0d6ebf3133d",
      "old_mode": 33188,
      "old_path": "tools/gn/function_write_file.cc",
      "new_id": "91599755a37e239ec3a4a155a8c46fb7459eda6b",
      "new_mode": 33188,
      "new_path": "tools/gn/function_write_file.cc"
    },
    {
      "type": "modify",
      "old_id": "9f19b2b0065321d12f74279d97dfbe492cf5ecc1",
      "old_mode": 33188,
      "old_path": "tools/gn/ninja_target_writer.cc",
      "new_id": "7d9c08afd65bdcb7741b52f8633ada63bfd56b79",
      "new_mode": 33188,
      "new_path": "tools/gn/ninja_target_writer.cc"
    },
    {
      "type": "modify",
      "old_id": "dec083753f79052cdc6a8a0ef80a75cfacf5d739",
      "old_mode": 33188,
      "old_path": "tools/gn/visual_studio_writer.cc",
      "new_id": "44a3f11449fd1efdd5be48a5e36d95bdf779f997",
      "new_mode": 33188,
      "new_path": "tools/gn/visual_studio_writer.cc"
    }
  ]
}
