Add some code to ensure that stdout is always flushed and closed to help debug layout tests hanging…

Add some code to ensure that stdout is always flushed and closed to help debug layout tests hanging on Windows trybots.

BUG=522396
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300202

[jam, 2016-04-26 17:59:33]

jam@chromium.org changed reviewers:
+ dpranke@chromium.org

[jam, 2016-04-26 17:59:34]

I'll try this for a day and then revert if it still occurs.

[jam, 2016-04-26 18:08:04]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-26 18:08:14]

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

[commit-bot: I haz the power, 2016-04-26 18:08:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-26 18:08:20]

Dry run: No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[Dirk Pranke, 2016-04-26 18:16:17]

lgtm

[jam, 2016-04-26 18:16:35]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2016-04-26 18:16:43]

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

[commit-bot: I haz the power, 2016-04-26 18:25:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-26 18:25:22]

Try jobs failed on following builders:
  Build Try Recipe Test Trusty64 on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Try%20Recipe%20...)

[jam, 2016-04-26 19:24:45]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2016-04-26 19:24:57]

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

[commit-bot: I haz the power, 2016-04-26 19:28:32]

Description was changed from

==========
Add some code to ensure that stdout is always flushed and closed to help debug
layout tests hanging on Windows trybots.

BUG=522396
==========

to

==========
Add some code to ensure that stdout is always flushed and closed to help debug
layout tests hanging on Windows trybots.

BUG=522396

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300202
==========

[commit-bot: I haz the power, 2016-04-26 19:28:33]

Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=300202

[jam, 2016-04-26 21:49:37]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1919403003/ by jam@chromium.org.

The reason for reverting is: hang still occurred:
https://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel....