[Android] Enable test_results_presentation for gtests

[Android] Enable test_results_presentation for gtests in chromium.linux.json.

This cl is for gtests on the android bots that are (still) on chromium.linux.

BUG=605572
Review-Url: https://codereview.chromium.org/2849343002
Cr-Commit-Position: refs/heads/master@{#469137}
Committed: https://chromium.googlesource.com/chromium/src/+/23935bc344576994ce5eb91be72a3907f7a68f3e

[BigBossZhiling, 2017-05-01 23:29:09]

Description was changed from

==========
Enable test_results_presentation for gtests in chromium.linux.json.

BUG=605572
==========

to

==========
Enable test_results_presentation for gtests in chromium.linux.json.

BUG=605572
==========

[BigBossZhiling, 2017-05-01 23:29:09]

hzl@google.com changed reviewers:
+ jbudorick@chromium.org

[jbudorick, 2017-05-03 17:11:37]

lgtm w/ nit

https://codereview.chromium.org/2849343002/diff/40001/build/android/pylib/res...
File build/android/pylib/results/presentation/test_results_presentation.py
(right):

https://codereview.chromium.org/2849343002/diff/40001/build/android/pylib/res...
build/android/pylib/results/presentation/test_results_presentation.py:156:
cell(data=result['status'] if result['status'] else 'UNKNOWN',
nit: data=result['status'] or 'UNKNOWN'

[BigBossZhiling, 2017-05-03 17:17:41]

https://codereview.chromium.org/2849343002/diff/40001/build/android/pylib/res...
File build/android/pylib/results/presentation/test_results_presentation.py
(right):

https://codereview.chromium.org/2849343002/diff/40001/build/android/pylib/res...
build/android/pylib/results/presentation/test_results_presentation.py:156:
cell(data=result['status'] if result['status'] else 'UNKNOWN',
On 2017/05/03 17:11:37, jbudorick wrote:
> nit: data=result['status'] or 'UNKNOWN'

Done.

[BigBossZhiling, 2017-05-03 17:17:44]

The CQ bit was checked by hzl@google.com

[BigBossZhiling, 2017-05-03 17:17:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org
Link to the patchset: https://codereview.chromium.org/2849343002/#ps60001
(title: "fixes")

[commit-bot: I haz the power, 2017-05-03 17:18:06]

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, 2017-05-03 17:28:41]

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

[commit-bot: I haz the power, 2017-05-03 17:28:42]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[BigBossZhiling, 2017-05-03 20:50:10]

hzl@google.com changed reviewers:
+ thakis@chromium.org

[BigBossZhiling, 2017-05-03 20:51:05]

Need owner approval. @thakis

[Nico, 2017-05-03 20:54:24]

Sorry, I lack context here. The linked bug links to a design doc that describes
how the flow is painful for android bots -- why are we enabling this on the
linux bots then?

[jbudorick, 2017-05-03 20:55:55]

On 2017/05/03 20:54:24, Nico wrote:
> Sorry, I lack context here. The linked bug links to a design doc that
describes
> how the flow is painful for android bots -- why are we enabling this on the
> linux bots then?

This is on the android bots that are (still) on chromium.linux.

[Nico, 2017-05-03 20:57:19]

lgtm with fixed cl description then.

[BigBossZhiling, 2017-05-03 21:03:18]

Description was changed from

==========
Enable test_results_presentation for gtests in chromium.linux.json.

BUG=605572
==========

to

==========
[Android] Enable test_results_presentation for gtests in chromium.linux.json.

This cl is for gtests on the android bots that are (still) on chromium.linux.

BUG=605572
==========

[BigBossZhiling, 2017-05-03 21:03:43]

The CQ bit was checked by hzl@google.com

[commit-bot: I haz the power, 2017-05-03 21:04:19]

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, 2017-05-03 21:41:59]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1493845423808780,
"parent_rev": "3f17b1463420e026459e2d7fde6335cf5e9b1852", "commit_rev":
"23935bc344576994ce5eb91be72a3907f7a68f3e"}

[commit-bot: I haz the power, 2017-05-03 21:42:18]

Description was changed from

==========
[Android] Enable test_results_presentation for gtests in chromium.linux.json.

This cl is for gtests on the android bots that are (still) on chromium.linux.

BUG=605572
==========

to

==========
[Android] Enable test_results_presentation for gtests in chromium.linux.json.

This cl is for gtests on the android bots that are (still) on chromium.linux.

BUG=605572

Review-Url: https://codereview.chromium.org/2849343002
Cr-Commit-Position: refs/heads/master@{#469137}
Committed:
https://chromium.googlesource.com/chromium/src/+/23935bc344576994ce5eb91be72a...
==========

[commit-bot: I haz the power, 2017-05-03 21:42:19]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/23935bc344576994ce5eb91be72a...