Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(79)

Side by Side Diff: docs/code_coverage.md

Issue 1306233003: Markdown style fixes for: (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: style changes Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « docs/cocoa_tips_and_tricks.md ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Categories of coverage 1 # Code Coverage
2 2
3 * <strong><font color='greeb'>executed</font></strong> - this line of code was hit during execution 3 ## Categories of coverage
4 * <strong><font color='orange'>instrumented</font></strong> - this line of cod e was part of the compilation unit, but not executed
5 * <strong><font color='red'>missing</font></strong> - in a source file, but no t compiled.
6 * ignored - not an executable line, or a line we don't care about
7 4
8 Coverage is calculated as `exe / (inst + miss)`. In general, lines that are in ` miss` should be ignored, but our exclusion rules are not good enough. 5 * executed - this line of code was hit during execution
6 * instrumented - this line of code was part of the compilation unit, but not
7 executed
8 * missing - in a source file, but not compiled.
9 * ignored - not an executable line, or a line we don't care about
9 10
10 # Buildbots 11 Coverage is calculated as `exe / (inst + miss)`. In general, lines that are in
12 `miss` should be ignored, but our exclusion rules are not good enough.
11 13
12 Buildbots are currently on the [experimental waterfall](http://build.chromium.or g/buildbot/waterfall.fyi/waterfall). The coverage figures they calculate come fr om running some subset of the chromium testing suite. 14 ## Buildbots
13 15
14 * [Linux](http://build.chromium.org/buildbot/waterfall.fyi/builders/Linux%20Co verage%20(dbg)) - uses `gcov` 16 Buildbots are currently on the
15 * [Windows](http://build.chromium.org/buildbot/waterfall.fyi/builders/Win%20Co verage%20%28dbg%29) 17 [experimental waterfall](http://build.chromium.org/buildbot/waterfall.fyi/waterf all).
16 * [Mac](http://build.chromium.org/buildbot/waterfall.fyi/builders/Mac%20Covera ge%20%28dbg%29) 18 The coverage figures they calculate come from running some subset of the
19 chromium testing suite.
20
21 * [Linux](http://build.chromium.org/buildbot/waterfall.fyi/builders/Linux%20Co verage%20(dbg))
22 - uses `gcov`
23 * [Windows](http://build.chromium.org/buildbot/waterfall.fyi/builders/Win%20Co verage%20%28dbg%29)
24 * [Mac](http://build.chromium.org/buildbot/waterfall.fyi/builders/Mac%20Covera ge%20%28dbg%29)
17 25
18 Also, 26 Also,
19 * [Coverage dashboard](http://build.chromium.org/buildbot/coverage/)
20 * [Example coverage summary](http://build.chromium.org/buildbot/coverage/linux -debug/49936/) - the coverage is calculated at directory and file level, and the directory structure is navigable via the **Subdirectories** table.
21 27
22 # Calculating coverage locally 28 * [Coverage dashboard](http://build.chromium.org/buildbot/coverage/)
29 * [Example coverage summary](http://build.chromium.org/buildbot/coverage/linux -debug/49936/)
30 - the coverage is calculated at directory and file level, and the directory
31 structure is navigable via the **Subdirectories** table.
32
33 ## Calculating coverage locally
23 34
24 TODO 35 TODO
25 36
26 # Advanced Tips 37 ## Advanced Tips
27 38
28 Sometimes a line of code should never be reached (e.g., `NOTREACHED()`). These c an be marked in the source with `// COV_NF_LINE`. Note that this syntax is exact . 39 Sometimes a line of code should never be reached (e.g., `NOTREACHED()`). These
40 can be marked in the source with `// COV_NF_LINE`. Note that this syntax is
41 exact.
OLDNEW
« no previous file with comments | « docs/cocoa_tips_and_tricks.md ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698