yup, looks buggy alright.

fixing this shouldn't block the migration, though (IMO) ...

Agreed that it shouldn't block migration. Thanks for fixing the tag.

Allow me to lobby as usual for only having outputs in the output dir. :/

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/562df4f33e0a29669b83d451d33c81648a9e023e

commit 562df4f33e0a29669b83d451d33c81648a9e023e
Author: brucedawson <brucedawson@chromium.org>
Date: Fri May 20 22:00:46 2016

Fix clobber.py to restore args.gn on failure

If delete_dir or os.mkdir throw an exception then the code to restore
args.gn, build.ninja, and build.ninja.d will not run. This breaks a
design goal of gn which is that the args.gn files should not be
deleted.

The fix is to catch the exception and raise it after the files have
been restored.

To test the code leave chrome.exe or base_unittests.exe running in one
window and run this in another:

    python build\clobber.py d:\src\chromium\src\out

Note that if you do this without the patch applied then all args.gn
files may be deleted, so consider backing them up. Also note that
args.gn files won't be restored unless build.ninja and build.ninja.d
files are also present, so back them up.

Just apply the patch first - it's easier.

BUG= 612940 

Review-Url: https://codereview.chromium.org/1996833002
Cr-Commit-Position: refs/heads/master@{#395177}

[modify] https://crrev.com/562df4f33e0a29669b83d451d33c81648a9e023e/build/clobber.py