Report test locations in the test launcher json summary.

Report test locations in the test launcher json summary.

Saving file/line of test declaration in machine-readable format allows
automatically find owners of broken/flacky tests.

BUG=

Committed: https://crrev.com/0e561e186e3eff8cf8eb123e115868ce8ecae6ee
Cr-Commit-Position: refs/heads/master@{#432462}

[alex-ac, 2016-11-15 13:50:00]

Description was changed from

==========
Report test locations in the test launcher json summary.

Saving file/line of test declaration in machine-readable format allows
automatically find owners of broken/flacky tests.

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

to

==========
Report test locations in the test launcher json summary.

Saving file/line of test declaration in machine-readable format allows
automatically find owners of broken/flacky tests.

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

[alex-ac, 2016-11-15 13:50:00]

alex-ac@yandex-team.ru changed reviewers:
+ phajdan.jr@chromium.org

[Paweł Hajdan Jr., 2016-11-16 08:13:58]

Thanks for the patch!

Do you have some effort related to flakiness? I'd be interested - also consider
talking to sergiyb@chromium.org .

https://codereview.chromium.org/2502953002/diff/1/base/test/launcher/test_res...
File base/test/launcher/test_results_tracker.cc (right):

https://codereview.chromium.org/2502953002/diff/1/base/test/launcher/test_res...
base/test/launcher/test_results_tracker.cc:386: for (auto& item :
test_locations_) {
nit: const auto&

[alex-ac, 2016-11-16 08:44:25]

> Do you have some effort related to flakiness? I'd be interested - also
consider
talking to sergiyb@chromium.org .

In yandex we have several hundreds of broken/flacky tests after each merge with
chromium dev channel (every 3 weeks). We use OWNERS of tests to find who must
fix test. Now we must use grep to find file in which test is declared. But this
patch allows to use test launcher summary for that.

https://codereview.chromium.org/2502953002/diff/1/base/test/launcher/test_res...
File base/test/launcher/test_results_tracker.cc (right):

https://codereview.chromium.org/2502953002/diff/1/base/test/launcher/test_res...
base/test/launcher/test_results_tracker.cc:386: for (auto& item :
test_locations_) {
On 2016/11/16 08:13:58, Paweł Hajdan Jr. wrote:
> nit: const auto&

Done.

[Paweł Hajdan Jr., 2016-11-16 09:25:26]

LGTM

[alex-ac, 2016-11-16 09:27:03]

The CQ bit was checked by alex-ac@yandex-team.ru

[commit-bot: I haz the power, 2016-11-16 09:27:15]

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-11-16 11:47:32]

Description was changed from

==========
Report test locations in the test launcher json summary.

Saving file/line of test declaration in machine-readable format allows
automatically find owners of broken/flacky tests.

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

to

==========
Report test locations in the test launcher json summary.

Saving file/line of test declaration in machine-readable format allows
automatically find owners of broken/flacky tests.

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

[commit-bot: I haz the power, 2016-11-16 11:47:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-16 11:49:31]

Description was changed from

==========
Report test locations in the test launcher json summary.

Saving file/line of test declaration in machine-readable format allows
automatically find owners of broken/flacky tests.

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

to

==========
Report test locations in the test launcher json summary.

Saving file/line of test declaration in machine-readable format allows
automatically find owners of broken/flacky tests.

BUG=

Committed: https://crrev.com/0e561e186e3eff8cf8eb123e115868ce8ecae6ee
Cr-Commit-Position: refs/heads/master@{#432462}
==========

[commit-bot: I haz the power, 2016-11-16 11:49:32]

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

[Sergiy Byelozyorov, 2016-11-16 12:09:00]

sergiyb@chromium.org changed reviewers:
+ sergiyb@chromium.org

[Sergiy Byelozyorov, 2016-11-16 12:09:00]

Wow! This is so cool. We have exactly same problem and there couldn't have been
better time for this CL to land. Thanks for much.