0
Commit Graph

17 Commits

Author SHA1 Message Date
Arthur Wang
7eb6a486fa Add two index to the sub titles.
Bug: b:360923880
Change-Id: I600ae66a2571c28487b4464c2225db08d5f41cd2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5817883
Commit-Queue: Arthur Wang <wuwang@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Reviewed-by: Prakhar Asthana <pasthana@google.com>
Cr-Commit-Position: refs/heads/main@{#1347583}
2024-08-27 20:10:09 +00:00
Arthur Wang
3cae925ddc Add a read artifact section on code coverage.
----
Add a section to explain the data format of the code coverage report.

Bug: b:360923880
Change-Id: I94d5ff02ae63584a0bf1d8f052aeacafdbd95e68

----
Update the code coverage documentation.

Change-Id: I94d5ff02ae63584a0bf1d8f052aeacafdbd95e68
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5817355
Reviewed-by: Peter Wen <wnwen@chromium.org>
Commit-Queue: Arthur Wang <wuwang@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1347571}
2024-08-27 19:47:23 +00:00
Alan Zhao
f8fa313044 Reland "Enable continuous mode/runtime counter relocation for coverage builds"
This is a reland of commit 0265879ffa

The original change was reverted because this enabled the continuous
mode on PGO bots, causing them to fail. The continuous mode was enabled
everywhere by the change made in the original CL to test_env.py. It is
believed that the continuous mode is incompatible with pgo builds
because PGO builds make manual calls to compiler-rt to flush the
counters. To address this, this reland wil omit the changes made to
test_env.py. Instead, continuous mode will be enabled on the bots later
by updating the recipe, which is in a separate repository.

Original change's description:
> Enable continuous mode/runtime counter relocation for coverage builds
>
> Enabling the continuous mode for coverage builds allows us to recover
> coverage data for test processes run in sandboxes while avoiding the CQ
> failures that we see with the noncontinuous mode that flushes all the
> counters to disk at process shutdown. This should also be expected to
> recover coverage data for tests that exit abnormally.
>
> Enabling the continuous mode is also the first step in landing several
> changes that will potentially improve instrumented test performance.
>
>
> Cq-Include-Trybots: luci.chromium.try:chromeos-amd64-generic-siso-rel
> Cq-Include-Trybots: luci.chromium.try:ios-simulator-siso
> Cq-Include-Trybots: luci.chromium.try:linux_chromium_asan_siso_rel_ng
> Cq-Include-Trybots: luci.chromium.try:mac-siso-rel
> Change-Id: Iab1177b2e8f5a24887a33417d1556c90f1002363
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4953786
> Reviewed-by: Junji Watanabe <jwata@google.com>
> Reviewed-by: danakj <danakj@chromium.org>
> Reviewed-by: Ben Pastene <bpastene@chromium.org>
> Reviewed-by: Hans Wennborg <hans@chromium.org>
> Commit-Queue: Alan Zhao <ayzhao@google.com>
> Reviewed-by: Dominic Farolino <dom@chromium.org>
> Reviewed-by: Prakhar Asthana <pasthana@google.com>
> Cr-Commit-Position: refs/heads/main@{#1235959}

Bug: 1462187,1468343,1510916
Change-Id: I8a065f0a5886a53f5509491796501dd5b288c99a
Cq-Include-Trybots: luci.chromium.try:chromeos-amd64-generic-siso-rel
Cq-Include-Trybots: luci.chromium.try:ios-simulator-siso
Cq-Include-Trybots: luci.chromium.try:linux_chromium_asan_siso_rel_ng
Cq-Include-Trybots: luci.chromium.try:mac-siso-rel
Cq-Include-Trybots: chrome/try:linux-pgo,mac-pgo,win64-pgo
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5122344
Commit-Queue: Prakhar Asthana <pasthana@google.com>
Reviewed-by: Junji Watanabe <jwata@google.com>
Reviewed-by: Prakhar Asthana <pasthana@google.com>
Reviewed-by: Dominic Farolino <dom@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Alan Zhao <ayzhao@google.com>
Cr-Commit-Position: refs/heads/main@{#1245683}
2024-01-11 03:35:39 +00:00
Kai Ninomiya
6f537ebb5e Update docs about code coverage on full Chromium
Running full Chromium with code coverage no longer requires disabling
the sandbox. It was fixed in 2019.

Found while working on crbug.com/1399354

Bug: None, 972232
No-Try: True
Change-Id: Ieeb86f162ebc0f955ace8d511ee3bf7c9b61cf92
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4129804
Reviewed-by: Zhaoyang Li <zhaoyangli@chromium.org>
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1089024}
2023-01-04 23:43:47 +00:00
Prakhar
f41864b0ca Add references to jacoco/clang coverage docs in each other
Bug: 1340699
Change-Id: Ia74ceced02342bac564a9233fb0b8c065eb65b40
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3760793
Auto-Submit: Prakhar Asthana <pasthana@google.com>
Reviewed-by: Zhaoyang Li <zhaoyangli@chromium.org>
Reviewed-by: Benjamin Joyce <bjoyce@chromium.org>
Commit-Queue: Benjamin Joyce <bjoyce@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1025347}
2022-07-18 19:03:51 +00:00
Prakhar
74514baf66 Fix link to coveage dashboard
Change-Id: Id0d0bc3355d9e369ac0f2e083c51c39ee1d664d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3654660
Auto-Submit: Prakhar Asthana <pasthana@google.com>
Reviewed-by: Zhaoyang Li <zhaoyangli@chromium.org>
Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1005275}
2022-05-19 17:14:04 +00:00
Prakhar
e84bb3b2b4 Update code coverage dashboard link
Change-Id: Ic65469f42fa06a85b66eac9272c0c91de3e86416
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3299742
Auto-Submit: Prakhar Asthana <pasthana@google.com>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/main@{#945841}
2021-11-29 03:28:44 +00:00
Choongwoo Han
567525224f Use -compilation-dir for llvm-cov
https://reviews.llvm.org/D100232

coverage builds now have relative paths to source code from the build
directory, so llvm-cov is failing to find the source code.
`-compilation-dir` is now available to specify the build directory
for the relative paths.

Bug: 1216203, 1204920
Change-Id: I79bfd6461c315fa9dc555c0829854e612685f122
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2950169
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#891281}
2021-06-10 17:38:34 +00:00
John Palmer
046f987e02 [COIL] Change chromium code search type links to use main rather than master for docs
See this doc: go/chromium-coil-change for more info
    BUG=1210385

Change-Id: I6cd8548301e360bae8e71384dbd8ef11aa6c985f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2902904
Reviewed-by: My Nguyen <myy@chromium.org>
Commit-Queue: My Nguyen <myy@chromium.org>
Auto-Submit: John Palmer <jopalmer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#885851}
2021-05-24 01:24:56 +00:00
Zequan Wu
6d00cccdc9 [Docs] Get llvm-cov and llvm-profdata by setting "checkout_clang_coverage_tools"
Change-Id: I219aad59ab54de07dc1206cf395a4b0f33821d7f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2761961
Auto-Submit: Zequan Wu <zequanwu@google.com>
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Zequan Wu <zequanwu@google.com>
Cr-Commit-Position: refs/heads/master@{#863006}
2021-03-15 22:40:44 +00:00
Ben Joyce
88282360b2 Update coverage for android native coverage.
Can run android code coverage locally with this change. Uses the same
command as  outlined:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/code_coverage.md#local-coverage-script

Bug: 985574
Change-Id: Icb0e9a1e55bfdad7f6a4bcb3c49a2abaa3e6729c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2658964
Commit-Queue: benjamin joyce <bjoyce@chromium.org>
Auto-Submit: benjamin joyce <bjoyce@chromium.org>
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#848784}
2021-01-29 23:53:31 +00:00
Ben Joyce
739fdb2083 Change quotation marks to be bash pastable.
Bug: None
Change-Id: I39c221fd574b9d19fe0d9cbabf81769644823e90
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2627142
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: benjamin joyce <bjoyce@chromium.org>
Cr-Commit-Position: refs/heads/master@{#843122}
2021-01-13 19:22:22 +00:00
Bryce Thomas
c9137cf998 Fix code_coverage.md docs to specify is_debug=false GN arg.
One instance of this was already fixed in
https://chromium-review.googlesource.com/c/chromium/src/+/1874486.  This
CL simply covers the other.

Bug: NONE
Change-Id: Ic0dc4ce2fdc1ca0daa414b50d2f9cf5a59f676ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130667
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Sajjad Mirza <sajjadm@chromium.org>
Commit-Queue: Sajjad Mirza <sajjadm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#755064}
2020-03-31 19:47:17 +00:00
Yuke Liao
8c0868fe61 [code coverage] Fix code coverage documentation
This CL fixes the code coverage documentation:
1. 'is_debug=false' gn arg is required.
2. Sandbox is working now.
3. Remove obsolete links.

Bug: 1012473
Change-Id: I178f42cd6879fbfaf464d344ee272268ee654ea3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1874486
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#708351}
2019-10-22 21:02:33 +00:00
Yuke Liao
03c644076d [code coverage] Update code coverage component references
The Monorail component for code coverage was renamed from:
Tools>CodeCoverage to Infra>Test>CodeCoverage, and this CL updates the
references.

Change-Id: I886422c5b8f839570ade415354fe2c122ee35ff7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726371
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#682373}
2019-07-30 18:33:40 +00:00
Yuke Liao
04efa97ad6 [code coverage] Update doc references to clang_coverage recipe module
The clang_coverage recipe module was renamed to code_coverage in
https://chromium-review.googlesource.com/c/chromium/tools/build/+/1670914.
This CL updates the references at chromium/src side accordingly.

Bug: 977032
Change-Id: I2059b978cbc75d3f8621e7c14880ee6d40a92f85
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1669984
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#672657}
2019-06-26 21:14:07 +00:00
Yuke Liao
43bbbcd535 [code coverage] Update documentation for code coverage
This CL updates the documentation for code coverage.

TBR=jbudorick@chromium.org

Bug: 974379
Change-Id: If9abde5ce40793a7a4d265703fb695126d8f0e56
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1663371
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Erik Staab <estaab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#671397}
2019-06-21 19:34:50 +00:00