Revert of [Android] Make the test runner terminate gracefully on SIGTERM.

Revert of [Android] Make the test runner terminate gracefully on SIGTERM. (patchset #2 id:20001 of https://codereview.chromium.org/2511733004/ )

Reason for revert:
Caused Android Builder to fail: https://uberchromegw.corp.google.com/i/chromium.webkit/builders/Android%20Builder/builds/97588/steps/compile/logs/stdio

Original issue's description:
> [Android] Make the test runner terminate gracefully on SIGTERM.
>
> This includes writing out the JSON results file, even if we have
> no results.
>
> BUG=664308
>
> Committed: https://crrev.com/b346038701a73dbc4c1758fe9baf7b23614fbe60
> Cr-Commit-Position: refs/heads/master@{#433234}

TBR=katthomas@google.com,rnephew@chromium.org,jbudorick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=664308
Committed: https://crrev.com/8b304e2f24383fc9e8d6c78f86d661ebbd3e1934
Cr-Commit-Position: refs/heads/master@{#433270}

[Fady Samuel, 2016-11-18 18:14:10]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-11-18 18:14:11]

Created Revert of [Android] Make the test runner terminate gracefully on
SIGTERM.

[commit-bot: I haz the power, 2016-11-18 18:14:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jbudorick, 2016-11-18 18:16:28]

The CQ bit was unchecked by jbudorick@chromium.org

[jbudorick, 2016-11-18 18:19:35]

On 2016/11/18 18:16:28, jbudorick wrote:
> The CQ bit was unchecked by mailto:jbudorick@chromium.org

I paused this temporarily to discuss.

[jbudorick, 2016-11-18 18:28:15]

On 2016/11/18 18:19:35, jbudorick wrote:
> On 2016/11/18 18:16:28, jbudorick wrote:
> > The CQ bit was unchecked by mailto:jbudorick@chromium.org
> 
> I paused this temporarily to discuss.

Bot rolled green independently. We'll land this revert if it fails again.

[jbudorick, 2016-11-18 19:42:22]

On 2016/11/18 18:28:15, jbudorick wrote:
> On 2016/11/18 18:19:35, jbudorick wrote:
> > On 2016/11/18 18:16:28, jbudorick wrote:
> > > The CQ bit was unchecked by mailto:jbudorick@chromium.org
> > 
> > I paused this temporarily to discuss.
> 
> Bot rolled green independently. We'll land this revert if it fails again.

bot failed again in similar fashion:
https://build.chromium.org/p/chromium.webkit/builders/Android%20Builder/build...

proceeding w/ speculative revert.

[jbudorick, 2016-11-18 19:42:26]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2016-11-18 19:43:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 19:44:01]

Description was changed from

==========
Revert of [Android] Make the test runner terminate gracefully on SIGTERM.
(patchset #2 id:20001 of https://codereview.chromium.org/2511733004/ )

Reason for revert:
Caused Android Builder to fail:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/Android%20Bui...


Original issue's description:
> [Android] Make the test runner terminate gracefully on SIGTERM.
> 
> This includes writing out the JSON results file, even if we have
> no results.
> 
> BUG=664308
> 
> Committed: https://crrev.com/b346038701a73dbc4c1758fe9baf7b23614fbe60
> Cr-Commit-Position: refs/heads/master@{#433234}

TBR=katthomas@google.com,rnephew@chromium.org,jbudorick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=664308
==========

to

==========
Revert of [Android] Make the test runner terminate gracefully on SIGTERM.
(patchset #2 id:20001 of https://codereview.chromium.org/2511733004/ )

Reason for revert:
Caused Android Builder to fail:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/Android%20Bui...


Original issue's description:
> [Android] Make the test runner terminate gracefully on SIGTERM.
> 
> This includes writing out the JSON results file, even if we have
> no results.
> 
> BUG=664308
> 
> Committed: https://crrev.com/b346038701a73dbc4c1758fe9baf7b23614fbe60
> Cr-Commit-Position: refs/heads/master@{#433234}

TBR=katthomas@google.com,rnephew@chromium.org,jbudorick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=664308
==========

[commit-bot: I haz the power, 2016-11-18 19:44:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-18 19:47:26]

Description was changed from

==========
Revert of [Android] Make the test runner terminate gracefully on SIGTERM.
(patchset #2 id:20001 of https://codereview.chromium.org/2511733004/ )

Reason for revert:
Caused Android Builder to fail:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/Android%20Bui...


Original issue's description:
> [Android] Make the test runner terminate gracefully on SIGTERM.
> 
> This includes writing out the JSON results file, even if we have
> no results.
> 
> BUG=664308
> 
> Committed: https://crrev.com/b346038701a73dbc4c1758fe9baf7b23614fbe60
> Cr-Commit-Position: refs/heads/master@{#433234}

TBR=katthomas@google.com,rnephew@chromium.org,jbudorick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=664308
==========

to

==========
Revert of [Android] Make the test runner terminate gracefully on SIGTERM.
(patchset #2 id:20001 of https://codereview.chromium.org/2511733004/ )

Reason for revert:
Caused Android Builder to fail:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/Android%20Bui...

Original issue's description:
> [Android] Make the test runner terminate gracefully on SIGTERM.
>
> This includes writing out the JSON results file, even if we have
> no results.
>
> BUG=664308
>
> Committed: https://crrev.com/b346038701a73dbc4c1758fe9baf7b23614fbe60
> Cr-Commit-Position: refs/heads/master@{#433234}

TBR=katthomas@google.com,rnephew@chromium.org,jbudorick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=664308

Committed: https://crrev.com/8b304e2f24383fc9e8d6c78f86d661ebbd3e1934
Cr-Commit-Position: refs/heads/master@{#433270}
==========

[commit-bot: I haz the power, 2016-11-18 19:47:27]

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

[jbudorick, 2016-11-18 20:21:23]

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

The reason for reverting is: After looking at that bot further, it's been
failing occasionally w/ similar issues for a while.

e.g.
https://build.chromium.org/p/chromium.webkit/builders/Android%20Builder/build...
https://build.chromium.org/p/chromium.webkit/builders/Android%20Builder/build...
https://build.chromium.org/p/chromium.webkit/builders/Android%20Builder/build...
https://build.chromium.org/p/chromium.webkit/builders/Android%20Builder/build...


I'll investigate that separately, but in the meantime, I'm relanding this..