Surface test times of android tests.

Surface test times of android tests.

Right now we have no easy visibility on time taken for individual tests, since everything runs under one massive "Sharded Perf Tests" step. We're trying to balance the tests across devices to improve cycle time, so we need a way to see how long each test is actually taking.

BUG=466101
Committed: https://crrev.com/8a9efca9b5451c75da244f62f073206c97ef6dd4
Cr-Commit-Position: refs/heads/master@{#329722}

[shatch, 2015-05-12 20:03:28]

simonhatch@chromium.org changed reviewers:
+ skyostil@chromium.org

[Sami, 2015-05-13 18:39:19]

lgtm.

https://codereview.chromium.org/1140783002/diff/1/build/android/pylib/perf/te...
File build/android/pylib/perf/test_runner.py (right):

https://codereview.chromium.org/1140783002/diff/1/build/android/pylib/perf/te...
build/android/pylib/perf/test_runner.py:87: 'total_time': -1}
Not sure if it makes more sense to set the total time to -1 or just leave out
'total_time' completely if we don't have results. I'll leave it to you to
decide.

[shatch, 2015-05-13 19:07:50]

On 2015/05/13 18:39:19, Sami wrote:
> lgtm.
> 
>
https://codereview.chromium.org/1140783002/diff/1/build/android/pylib/perf/te...
> File build/android/pylib/perf/test_runner.py (right):
> 
>
https://codereview.chromium.org/1140783002/diff/1/build/android/pylib/perf/te...
> build/android/pylib/perf/test_runner.py:87: 'total_time': -1}
> Not sure if it makes more sense to set the total time to -1 or just leave out
> 'total_time' completely if we don't have results. I'll leave it to you to
> decide.

Hmm yeah might look a little cleaner just leaving it out, either way it'll just
be a small check.

[shatch, 2015-05-13 19:07:56]

The CQ bit was checked by simonhatch@chromium.org

[shatch, 2015-05-13 19:07:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/1140783002/#ps20001
(title: "Leave out total_time.")

[commit-bot: I haz the power, 2015-05-13 19:09:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1140783002/20001

[commit-bot: I haz the power, 2015-05-13 22:00:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-13 22:02:19]

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