Add link to flakiness dashboard for each test case

Add link to flakiness dashboard for each test case

Create a link to flakiness dashboard for each test case, so that engineers will see the flakiness level for that test case.

BUG=717631
Review-Url: https://codereview.chromium.org/2873723002
Cr-Commit-Position: refs/heads/master@{#471122}
Committed: https://chromium.googlesource.com/chromium/src/+/20892f4eb9dad7ed0b1720dc3718dbc5db5f0d7f

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

Description was changed from

==========
Sample crashed testcases.

BUG=
==========

to

==========
Add link to flakiness dashboard for each test case

Create a link to flakiness dashboard for each test case, so that engineers will
see the flakiness level for that test case.

BUG=717631
==========

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

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

[jbudorick, 2017-05-11 18:21:08]

lgtm w/ nits

https://codereview.chromium.org/2873723002/diff/120001/build/android/pylib/re...
File build/android/pylib/results/presentation/test_results_presentation.py
(right):

https://codereview.chromium.org/2873723002/diff/120001/build/android/pylib/re...
build/android/pylib/results/presentation/test_results_presentation.py:117:
data='flakiness',
nit: 4 space indent

https://codereview.chromium.org/2873723002/diff/120001/build/android/pylib/re...
build/android/pylib/results/presentation/test_results_presentation.py:174:
cell(data=result['elapsed_time_ms']),   # elapsed_time_ms
nit: if we're going to have trailing comments lined up like this, you ought to
realign them to match the line below. Alternatively, you could put them on the
preceding line.

[jbudorick, 2017-05-11 18:21:30]

https://codereview.chromium.org/2873723002/diff/120001/tools/mb/mb.py
File tools/mb/mb.py (right):

https://codereview.chromium.org/2873723002/diff/120001/tools/mb/mb.py#newcode...
tools/mb/mb.py:1095: '--store-tombstones']
actually, this shouldn't be necessary for flakiness dashboard links?

[BigBossZhiling, 2017-05-11 21:47:51]

The CQ bit was checked by hzl@google.com

[BigBossZhiling, 2017-05-11 21:47:51]

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/2873723002/#ps140001
(title: "fixes")

[BigBossZhiling, 2017-05-11 21:47:56]

https://codereview.chromium.org/2873723002/diff/120001/build/android/pylib/re...
File build/android/pylib/results/presentation/test_results_presentation.py
(right):

https://codereview.chromium.org/2873723002/diff/120001/build/android/pylib/re...
build/android/pylib/results/presentation/test_results_presentation.py:117:
data='flakiness',
On 2017/05/11 18:21:08, jbudorick wrote:
> nit: 4 space indent

Done.

https://codereview.chromium.org/2873723002/diff/120001/build/android/pylib/re...
build/android/pylib/results/presentation/test_results_presentation.py:174:
cell(data=result['elapsed_time_ms']),   # elapsed_time_ms
On 2017/05/11 18:21:08, jbudorick wrote:
> nit: if we're going to have trailing comments lined up like this, you ought to
> realign them to match the line below. Alternatively, you could put them on the
> preceding line.

Done.

https://codereview.chromium.org/2873723002/diff/120001/tools/mb/mb.py
File tools/mb/mb.py (right):

https://codereview.chromium.org/2873723002/diff/120001/tools/mb/mb.py#newcode...
tools/mb/mb.py:1095: '--store-tombstones']
On 2017/05/11 18:21:30, jbudorick wrote:
> actually, this shouldn't be necessary for flakiness dashboard links?

Done.

[commit-bot: I haz the power, 2017-05-11 21:49:25]

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-11 23:45:26]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1494539271082890,
"parent_rev": "0762cfd079b91ff195345470c6253e9605716e0c", "commit_rev":
"20892f4eb9dad7ed0b1720dc3718dbc5db5f0d7f"}

[commit-bot: I haz the power, 2017-05-11 23:45:35]

Description was changed from

==========
Add link to flakiness dashboard for each test case

Create a link to flakiness dashboard for each test case, so that engineers will
see the flakiness level for that test case.

BUG=717631
==========

to

==========
Add link to flakiness dashboard for each test case

Create a link to flakiness dashboard for each test case, so that engineers will
see the flakiness level for that test case.

BUG=717631

Review-Url: https://codereview.chromium.org/2873723002
Cr-Commit-Position: refs/heads/master@{#471122}
Committed:
https://chromium.googlesource.com/chromium/src/+/20892f4eb9dad7ed0b1720dc3718...
==========

[commit-bot: I haz the power, 2017-05-11 23:45:36]

Committed patchset #8 (id:140001) as
https://chromium.googlesource.com/chromium/src/+/20892f4eb9dad7ed0b1720dc3718...