Skip to content

Commit

Permalink
[code coverage] Update doc references to clang_coverage recipe module
Browse files Browse the repository at this point in the history
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}
  • Loading branch information
Yuke Liao authored and Commit Bot committed Jun 26, 2019
1 parent b9b3e9e commit 04efa97
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/testing/code_coverage.md
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,7 @@ because of significant shared logic.

The code used by the bots that generate the coverage data lives (among other
places) in the
[clang coverage recipe module](https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave/recipe_modules/clang_coverage/).
[code coverage recipe module](https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave/recipe_modules/code_coverage/).

### Why is coverage for X not reported or unreasonably low, even though there is a test for X?

Expand Down
2 changes: 1 addition & 1 deletion tools/mb/mb_config.pyl
Original file line number Diff line number Diff line change
Expand Up @@ -2281,7 +2281,7 @@

# Used to pass the list of files to instrument for coverage to the compile
# wrapper. See:
# https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/clang_coverage/api.py
# https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/code_coverage/api.py
# and
# https://cs.chromium.org/chromium/src/docs/clang_code_coverage_wrapper.md
'partial_clang_instrumentation': {
Expand Down

0 comments on commit 04efa97

Please sign in to comment.