This is an archive of the discontinued LLVM Phabricator instance.

Differential D47231

[Coverage] Update CSS to make HTML reports copy-paste friendly.
ClosedPublic

Authored by Dor1s on May 22 2018, 2:54 PM.

Details

Reviewers
vsk
Commits
rG5b1083d76f38: [Coverage] Update CSS to make HTML reports copy-paste friendly.
rL333034: [Coverage] Update CSS to make HTML reports copy-paste friendly.
Summary

This minor change allows to copy snippets from HTML reports so they
will be pasted in the following format:
%LineNumber%\t%HitCount%\t%CodeLine%

rather then being split onto multiple lines. To see this in action, try copy
pasting from https://chromium-coverage.appspot.com/reports/560344/linux/chromium/src/third_party/zlib/compress.c.html

Requested in https://bugs.chromium.org/p/chromium/issues/detail?id=845571

Diff Detail

Event Timeline

Dor1s created this revision.May 22 2018, 2:54 PM
Herald added a subscriber: llvm-commits. · View Herald TranscriptMay 22 2018, 2:54 PM
Harbormaster completed remote builds in B18480: Diff 148109.May 22 2018, 2:54 PM
Dor1s edited the summary of this revision. (Show Details)May 22 2018, 2:55 PM
Dor1s edited the summary of this revision. (Show Details)May 22 2018, 2:57 PM
vsk accepted this revision.May 22 2018, 3:07 PM

LGTM. Thanks!

This revision is now accepted and ready to land.May 22 2018, 3:07 PM
Closed by commit rL333034: [Coverage] Update CSS to make HTML reports copy-paste friendly. (authored by Dor1s). · Explain WhyMay 22 2018, 3:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
tools/
llvm-cov/
3 lines

Diff 148109

tools/llvm-cov/SourceCoverageViewHTML.cpp

Show First 20 Lines • Show All 199 Lines • ▼ Show 20 Lines
th, td { th, td {
vertical-align: top; vertical-align: top;
padding: 2px 8px; padding: 2px 8px;
border-collapse: collapse; border-collapse: collapse;
border-right: solid 1px #eee; border-right: solid 1px #eee;
border-left: solid 1px #eee; border-left: solid 1px #eee;
text-align: left; text-align: left;
} }
td pre {
display: inline-block;
}
td:first-child { td:first-child {
border-left: none; border-left: none;
} }
td:last-child { td:last-child {
border-right: none; border-right: none;
} }
tr:hover { tr:hover {
background-color: #f0f0f0; background-color: #f0f0f0;
▲ Show 20 Lines • Show All 482 Lines • Show Last 20 Lines