Closed Bug 1036449 Opened 6 years ago Closed 6 years ago

CSS Coverage start should begin with a page refresh

Categories

(DevTools :: Style Editor, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 33

People

(Reporter: jwalker, Assigned: jwalker)

Details

Attachments

(1 file)

Downsides:

* Refreshing a page might be annoying in some cases
  But just about to start a coverage test probably isn't one of those

Upsides:

* Coverage will work the way people expect more
What about the use that I start css coverage, visit 3-4 diff. pages which have the same CSS and then stop the coverage to know the real coverage of my CSS file.

This is a very good use case as many people share the CSS files across pages and the file is not completely used one each single page, but collectively, the file's usage is almost 100%.

Devs might want to figure out the extra things that make it almost 100% rather than 100%
(In reply to Girish Sharma [:Optimizer] from comment #1)
> What about the use that I start css coverage, visit 3-4 diff. pages which
> have the same CSS and then stop the coverage to know the real coverage of my
> CSS file.

That requirement is supported, and isn't changed by this bug.
So after this bug, the csscoverage report won't restart after every (re)load ? but will only start after the first reload ?
(In reply to Girish Sharma [:Optimizer] from comment #3)
> So after this bug, the csscoverage report won't restart after every (re)load
> ? but will only start after the first reload ?

No. It's saying the pressing the start button reloads the current page.
Ah, my bad.
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Attachment #8454453 - Flags: review?(fayearthur)
Attachment #8454453 - Flags: review?(fayearthur) → review+
https://hg.mozilla.org/mozilla-central/rev/33f1bc57b49c
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 33
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.