Print actual list of failing tests on Android layout test steps

Print actual list of failing tests on Android layout test steps

This changes the Android bots to mimic the behavior of the non-Android bots
and to print the actual list of failing/flaky/unexpected passing tests
as part of the step text.

R=ilevy@chromium.org
BUG=293920
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=227850

[Dirk Pranke, 2013-10-07 23:28:01]

adding yet a third (4th) version of this code in a different place :) ...

[Isaac (away), 2013-10-09 18:27:59]

lgtm w/ nit

https://codereview.chromium.org/26330004/diff/1/build/android/buildbot/bb_dev...
File build/android/buildbot/bb_device_steps.py (right):

https://codereview.chromium.org/26330004/diff/1/build/android/buildbot/bb_dev...
build/android/buildbot/bb_device_steps.py:270: unexpected_failures,
unexpected_flakes, unexpected_passes = \
Extend lines using parens, not backslash, per google python conventions.

Could you run this through gpylint and see if anything else obvious comes up?

[Dirk Pranke, 2013-10-09 19:02:32]

On 2013/10/09 18:27:59, Isaac wrote:
> lgtm w/ nit
> 
>
https://codereview.chromium.org/26330004/diff/1/build/android/buildbot/bb_dev...
> File build/android/buildbot/bb_device_steps.py (right):
> 
>
https://codereview.chromium.org/26330004/diff/1/build/android/buildbot/bb_dev...
> build/android/buildbot/bb_device_steps.py:270: unexpected_failures,
> unexpected_flakes, unexpected_passes = \
> Extend lines using parens, not backslash, per google python conventions.
> 

Good catch. Fixed.

> Could you run this through gpylint and see if anything else obvious comes up?

The presubmit hooks lint the file and didn't catch anything else (the above
wasn't caught, either).

[commit-bot: I haz the power, 2013-10-09 19:04:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dpranke@chromium.org/26330004/5001

[commit-bot: I haz the power, 2013-10-09 19:48:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dpranke@chromium.org/26330004/10001

[commit-bot: I haz the power, 2013-10-10 00:46:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dpranke@chromium.org/26330004/10001

[commit-bot: I haz the power, 2013-10-10 01:11:27]

Change committed as 227850