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

Issue 833912 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 759794



Sign in to add a comment

Link documentation and the dashboard and the coverage script

Project Member Reported by mmoroz@chromium.org, Apr 17 2018

Issue description

subj
 

Comment 1 by mmoroz@chromium.org, Apr 17 2018

Labels: -Pri-3 Pri-0
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 18 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8e209fe889143d543540e3a3c1e4c0e2751e6d7b

commit 8e209fe889143d543540e3a3c1e4c0e2751e6d7b
Author: Yuke Liao <liaoyuke@chromium.org>
Date: Wed Apr 18 20:36:38 2018

[Coverage] Link to coverage.py to code_coverage.md

Link to coverage.py to code_coverage.md

Bug:  833912 
Change-Id: I089bbef64424004f543823d663dad00b3a275fbe
Reviewed-on: https://chromium-review.googlesource.com/1017239
Reviewed-by: Max Moroz <mmoroz@chromium.org>
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551809}
[modify] https://crrev.com/8e209fe889143d543540e3a3c1e4c0e2751e6d7b/tools/code_coverage/coverage.py

Owner: mmoroz@chromium.org
Status: Assigned (was: Untriaged)
Assigning to Max to fix the dashboard part.

Comment 4 by mmoroz@chromium.org, Apr 18 2018

Labels: -Pri-0 Pri-1
Thanks Yuke. I've also added links to the existing dashboard (http://go/chrome-coverage or https://chrome-coverage.storage.googleapis.com/index.html), so lowering the priority here but keeping it open for a little while once I finalize a URL hosting reports and add it everywhere as well.

Comment 5 by mmoroz@chromium.org, Apr 26 2018

Labels: Coverage-v1-Blocker

Comment 6 by mmoroz@chromium.org, Apr 26 2018

Blocking: 759794
Project Member

Comment 7 by bugdroid1@chromium.org, May 8 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/code-coverage/+/3bc3352c5eb31ba62a99eac3cb91b0106d10271f

commit 3bc3352c5eb31ba62a99eac3cb91b0106d10271f
Author: Max Moroz <mmoroz@google.com>
Date: Tue May 08 19:50:25 2018

Comment 8 by mmoroz@chromium.org, May 11 2018

The latest CL for this issue is in progress: https://chromium-review.googlesource.com/c/chromium/src/+/1054425

Let me mark this as fixed just to avoid forgetting to close it next week + to have a cleaner list of all known issues.

Comment 9 by mmoroz@chromium.org, May 11 2018

Status: Fixed (was: Assigned)
Project Member

Comment 10 by bugdroid1@chromium.org, May 11 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/af9811f265be48e4be8961580258c3aebcfdef84

commit af9811f265be48e4be8961580258c3aebcfdef84
Author: Abhishek Arya <inferno@chromium.org>
Date: Fri May 11 22:17:48 2018

Coverage: doc improvements.

R=mmoroz@chromium.org,liaoyuke@chromium.org

Change-Id: Iaf6f74ab4bc02d95f93f0ffc44ccf20e0d1f0682
Bug:  833912 
Reviewed-on: https://chromium-review.googlesource.com/1056153
Commit-Queue: Abhishek Arya <inferno@chromium.org>
Reviewed-by: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558035}
[modify] https://crrev.com/af9811f265be48e4be8961580258c3aebcfdef84/docs/code_coverage.md
[add] https://crrev.com/af9811f265be48e4be8961580258c3aebcfdef84/docs/images/code_coverage_source_view.png

Project Member

Comment 11 by bugdroid1@chromium.org, May 15 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3a9289055e78c6580b0debbd96ddfab89b336f4d

commit 3a9289055e78c6580b0debbd96ddfab89b336f4d
Author: Max Moroz <mmoroz@chromium.org>
Date: Tue May 15 14:39:50 2018

[Coverage] Add more FAQ entries to the doc + add link to the dashboard.

Bug:  833912 
Change-Id: Ie6bff55a2f3af2c670367315efd21d063e0c317d
Reviewed-on: https://chromium-review.googlesource.com/1054425
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Reviewed-by: Abhishek Arya <inferno@chromium.org>
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Jonathan Metzman <metzman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558700}
[modify] https://crrev.com/3a9289055e78c6580b0debbd96ddfab89b336f4d/docs/code_coverage.md

Sign in to add a comment