Fix isolate_tests after https://codereview.chromium.org/1825723002

Fix isolate_tests after https://codereview.chromium.org/1825723002

That change added display_unittests on swarming to several testing/buildbot
json files, but didn't update gn_migration.gypi to make sure
display_unittests_run runs on all platforms (so when the test was prepared
for swarming, the corresponding .isolated file might not yet have been
created).

Should fix e.g. the red isolate_tests step on
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMacASan/builds/5203

BUG=597105
TBR=robliao

Committed: https://crrev.com/9f9318e2cf3a367967014a354e0d280bdf66247c
Cr-Commit-Position: refs/heads/master@{#383963}

[Nico, 2016-03-30 11:58:37]

Description was changed from

==========
Fix isolate_tests after https://codereview.chromium.org/1825723002

That change added display_unittests on swarming to several testing/buildbot
json files, but didn't update gn_migration.gypi to make sure
display_unittests_run runs on all platforms (so when the test was prepared
for swarming, the corresponding .isolated file might not yet have been
created).

Should fix e.g. the red isolate_tests step on
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMacASan/builds/5203

BUG=597105
==========

to

==========
Fix isolate_tests after https://codereview.chromium.org/1825723002

That change added display_unittests on swarming to several testing/buildbot
json files, but didn't update gn_migration.gypi to make sure
display_unittests_run runs on all platforms (so when the test was prepared
for swarming, the corresponding .isolated file might not yet have been
created).

Should fix e.g. the red isolate_tests step on
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMacASan/builds/5203

BUG=597105
TBR=robliao
==========

[Nico, 2016-03-30 11:59:35]

thakis@chromium.org changed reviewers:
+ robliao@chromium.org

[Nico, 2016-03-30 11:59:36]

(took me a bit longer to find than necessary because
https://codereview.chromium.org/1825723002's CL description doesn't say anything
about letting the test run on more bots. Also, adding the test on more bots
seems fairly unrelated to the rest of that CL and should've probably been in a
separate change?)

[Nico, 2016-03-30 11:59:39]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-03-30 11:59:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1836353007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1836353007/1

[commit-bot: I haz the power, 2016-03-30 13:18:39]

Description was changed from

==========
Fix isolate_tests after https://codereview.chromium.org/1825723002

That change added display_unittests on swarming to several testing/buildbot
json files, but didn't update gn_migration.gypi to make sure
display_unittests_run runs on all platforms (so when the test was prepared
for swarming, the corresponding .isolated file might not yet have been
created).

Should fix e.g. the red isolate_tests step on
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMacASan/builds/5203

BUG=597105
TBR=robliao
==========

to

==========
Fix isolate_tests after https://codereview.chromium.org/1825723002

That change added display_unittests on swarming to several testing/buildbot
json files, but didn't update gn_migration.gypi to make sure
display_unittests_run runs on all platforms (so when the test was prepared
for swarming, the corresponding .isolated file might not yet have been
created).

Should fix e.g. the red isolate_tests step on
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMacASan/builds/5203

BUG=597105
TBR=robliao
==========

[commit-bot: I haz the power, 2016-03-30 13:18:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-30 13:19:45]

Description was changed from

==========
Fix isolate_tests after https://codereview.chromium.org/1825723002

That change added display_unittests on swarming to several testing/buildbot
json files, but didn't update gn_migration.gypi to make sure
display_unittests_run runs on all platforms (so when the test was prepared
for swarming, the corresponding .isolated file might not yet have been
created).

Should fix e.g. the red isolate_tests step on
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMacASan/builds/5203

BUG=597105
TBR=robliao
==========

to

==========
Fix isolate_tests after https://codereview.chromium.org/1825723002

That change added display_unittests on swarming to several testing/buildbot
json files, but didn't update gn_migration.gypi to make sure
display_unittests_run runs on all platforms (so when the test was prepared
for swarming, the corresponding .isolated file might not yet have been
created).

Should fix e.g. the red isolate_tests step on
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMacASan/builds/5203

BUG=597105
TBR=robliao

Committed: https://crrev.com/9f9318e2cf3a367967014a354e0d280bdf66247c
Cr-Commit-Position: refs/heads/master@{#383963}
==========

[commit-bot: I haz the power, 2016-03-30 13:19:46]

Patchset 1 (id:??) landed as
https://crrev.com/9f9318e2cf3a367967014a354e0d280bdf66247c
Cr-Commit-Position: refs/heads/master@{#383963}

[robliao, 2016-03-30 17:11:00]

On 2016/03/30 13:19:46, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/9f9318e2cf3a367967014a354e0d280bdf66247c
> Cr-Commit-Position: refs/heads/master@{#383963}

I'll have to separate the changes for next time. Adding them made sense at the
time to get trybot coverage.
I guess the CQ dry-run gave a false sense of confidence succeeding. I wonder if
there's a sanity check way to verify the configs before committing.