New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 799502 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task

Blocking:
issue 784464



Sign in to add a comment

Decide what to display in the index.html of code coverage reports.

Project Member Reported by liaoyuke@chromium.org, Jan 5 2018

Issue description

According to Max's CL comments:

"""
I don't have a strong opinion here, but have a few suggestions. We can discuss it and decide later, current implementation looks good.

1) index.html file shows per directory coverage, but also has a link at the top, something like "Click here to see coverage report for all source files" (needs a better phrasing I guess)

2) index.html file shows per source file coverage, but has a link at the top, something like "Click here to see per directory coverage". I like this less than the 1st, as coverage per directory seems to be more useful in a general case

3) index.html file has two links to different pages: to per directory coverage and to per file
"""

It needs more investigations to understand what's best to display in the index.html file of coverage report for better user experience.
 
Blocking: 784464

Comment 2 by mmoroz@chromium.org, Feb 26 2018

Status: Fixed (was: Assigned)
Current implementation looks good to me, as it shows per-directory summary by default, but also allows to switch to per-component or per-file view. Marking as Fixed, please re-open if more work is needed.

Sign in to add a comment