[coverage] Always merge sancov files after testing.

[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
NOTRY=true

Committed: https://crrev.com/74282397e4f05140564a1f86e364003cbc21eeaf
Cr-Commit-Position: refs/heads/master@{#34606}

[Michael Achenbach, 2016-03-08 19:54:30]

Description was changed from

==========
[coverage] Always merge sancov files after testing.

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

to

==========
[coverage] Always merge sancov files after testing.

BUG=chromium:568949
LOG=n
==========

[Michael Achenbach, 2016-03-08 19:54:30]

machenbach@chromium.org changed reviewers:
+ kjellander@chromium.org, tandrii@chromium.org

[Michael Achenbach, 2016-03-08 19:57:57]

Description was changed from

==========
[coverage] Always merge sancov files after testing.

BUG=chromium:568949
LOG=n
==========

to

==========
[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
==========

[Michael Achenbach, 2016-03-08 19:58:08]

PTAL

[kjellander_chromium, 2016-03-08 20:30:53]

lgtm with a suggestion

https://codereview.chromium.org/1776123002/diff/1/tools/run-tests.py
File tools/run-tests.py (right):

https://codereview.chromium.org/1776123002/diff/1/tools/run-tests.py#newcode835
tools/run-tests.py:835: 
Maybe you'd want to use check_call and try+except instead and print a
descriptive error message in case it fails?

[tandrii(chromium), 2016-03-08 21:44:54]

LGTM % same as Henrik's comment though given that call() doesn't eat
stdout/stderr, i believe the reason would still be visible in logs.

[Michael Achenbach, 2016-03-09 08:27:07]

Done

[Michael Achenbach, 2016-03-09 08:44:19]

Description was changed from

==========
[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
==========

to

==========
[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
NOTRY=true
==========

[Michael Achenbach, 2016-03-09 08:44:24]

The CQ bit was checked by machenbach@chromium.org

[Michael Achenbach, 2016-03-09 08:44:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
kjellander@chromium.org, tandrii@chromium.org
Link to the patchset: https://codereview.chromium.org/1776123002/#ps20001
(title: "Review")

[commit-bot: I haz the power, 2016-03-09 08:44:33]

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

[commit-bot: I haz the power, 2016-03-09 08:47:00]

Description was changed from

==========
[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
NOTRY=true
==========

to

==========
[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
NOTRY=true
==========

[commit-bot: I haz the power, 2016-03-09 08:47:01]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-09 08:48:01]

Description was changed from

==========
[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
NOTRY=true
==========

to

==========
[coverage] Always merge sancov files after testing.

With this, the test runner automatically merges sancov
files after testing. There's no need to do this by some
external infrastructure.

In a future CL, we could even merge during testing to lift
harddisk pressure.

BUG=chromium:568949
LOG=n
NOTRY=true

Committed: https://crrev.com/74282397e4f05140564a1f86e364003cbc21eeaf
Cr-Commit-Position: refs/heads/master@{#34606}
==========

[commit-bot: I haz the power, 2016-03-09 08:48:02]

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