[Android] Add heartbeat of platform mode perf test runner.

[Android] Add heartbeat of platform mode perf test runner.

Extremely long running tests can cause timeouts due to lack of output. This
adds a heartbeat message to stop that while tests are still running.

BUG=615157, 630640
Committed: https://crrev.com/e8555d3785393f8bf04579226a9e39f4e574c599
Cr-Commit-Position: refs/heads/master@{#407576}

[rnephew (Reviews Here), 2016-07-25 16:51:16]

Patchset #1 (id:1) has been deleted

[rnephew (Reviews Here), 2016-07-25 16:51:33]

rnephew@chromium.org changed reviewers:
+ jbudorick@chromium.org, mikecase@chromium.org

[mikecase (-- gone --), 2016-07-25 17:01:28]

cool, lgtm. This seems more general than to just perf tests though. So might
want to consider moving to the base class if you think that would be at all
beneficial.

https://codereview.chromium.org/2182563002/diff/20001/build/android/pylib/loc...
File build/android/pylib/local/device/local_device_perf_test_run.py (right):

https://codereview.chromium.org/2182563002/diff/20001/build/android/pylib/loc...
build/android/pylib/local/device/local_device_perf_test_run.py:33: def
__init__(self, shard, wait_time=60*10):
nit: Add some documentation. Specifically, to just say what units wait_time is
in.

[rnephew (Reviews Here), 2016-07-25 17:06:42]

https://codereview.chromium.org/2182563002/diff/20001/build/android/pylib/loc...
File build/android/pylib/local/device/local_device_perf_test_run.py (right):

https://codereview.chromium.org/2182563002/diff/20001/build/android/pylib/loc...
build/android/pylib/local/device/local_device_perf_test_run.py:33: def
__init__(self, shard, wait_time=60*10):
On 2016/07/25 17:01:27, mikecase wrote:
> nit: Add some documentation. Specifically, to just say what units wait_time is
> in.

Done.

[rnephew (Reviews Here), 2016-07-25 18:35:31]

Description was changed from

==========
[Android] Add heartbeat of platform mode perf test runner.

Extremely long running tests can cause timeouts due to lack of output. This
adds a heartbeat message to stop that while tests are still running.

BUG=615157
==========

to

==========
[Android] Add heartbeat of platform mode perf test runner.

Extremely long running tests can cause timeouts due to lack of output. This
adds a heartbeat message to stop that while tests are still running.

BUG=615157, 630640
==========

[jbudorick, 2016-07-25 19:57:28]

lgtm w/ optional stuff

https://codereview.chromium.org/2182563002/diff/40001/build/android/pylib/loc...
File build/android/pylib/local/device/local_device_perf_test_run.py (right):

https://codereview.chromium.org/2182563002/diff/40001/build/android/pylib/loc...
build/android/pylib/local/device/local_device_perf_test_run.py:31: class
HeartBeat(object):
It'd be nice if we could reconcile this w/ the existing heartbeat logging in
devil, but I suppose that doesn't need to block this.

[rnephew (Reviews Here), 2016-07-25 19:58:52]

The CQ bit was checked by rnephew@chromium.org

[rnephew (Reviews Here), 2016-07-25 19:58:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
mikecase@chromium.org
Link to the patchset: https://codereview.chromium.org/2182563002/#ps40001
(title: "add comments")

[commit-bot: I haz the power, 2016-07-25 19:59:18]

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-07-25 21:18:03]

Description was changed from

==========
[Android] Add heartbeat of platform mode perf test runner.

Extremely long running tests can cause timeouts due to lack of output. This
adds a heartbeat message to stop that while tests are still running.

BUG=615157, 630640
==========

to

==========
[Android] Add heartbeat of platform mode perf test runner.

Extremely long running tests can cause timeouts due to lack of output. This
adds a heartbeat message to stop that while tests are still running.

BUG=615157, 630640
==========

[commit-bot: I haz the power, 2016-07-25 21:18:04]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-07-25 21:20:00]

Description was changed from

==========
[Android] Add heartbeat of platform mode perf test runner.

Extremely long running tests can cause timeouts due to lack of output. This
adds a heartbeat message to stop that while tests are still running.

BUG=615157, 630640
==========

to

==========
[Android] Add heartbeat of platform mode perf test runner.

Extremely long running tests can cause timeouts due to lack of output. This
adds a heartbeat message to stop that while tests are still running.

BUG=615157, 630640

Committed: https://crrev.com/e8555d3785393f8bf04579226a9e39f4e574c599
Cr-Commit-Position: refs/heads/master@{#407576}
==========

[commit-bot: I haz the power, 2016-07-25 21:20:02]

Patchset 2 (id:??) landed as
https://crrev.com/e8555d3785393f8bf04579226a9e39f4e574c599
Cr-Commit-Position: refs/heads/master@{#407576}