Closed
Bug 1016285
Opened 10 years ago
Closed 10 years ago
CSS coverage needs more unit tests
Categories
(DevTools :: Style Editor, defect)
DevTools
Style Editor
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 33
People
(Reporter: jwalker, Assigned: jwalker)
References
Details
Attachments
(1 file)
26.91 KB,
patch
|
harth
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Should fix and have tests for bug 1016330 comment 1 as part of this. "Some rules are considered as used when they're not. For example: http://i.imgur.com/kFwydII.png A querySelectorAll of the 2nd rule doesn't return any element in any loaded document."
Assignee | ||
Comment 2•10 years ago
|
||
The changes to csscoverage.js are broken out here: https://github.com/joewalker/gecko-dev/pull/6
Assignee | ||
Comment 3•10 years ago
|
||
Green: https://tbpl.mozilla.org/?tree=Try&rev=1985f5fa0584
Updated•10 years ago
|
Attachment #8438561 -
Flags: review?(fayearthur) → review+
Assignee | ||
Comment 4•10 years ago
|
||
https://tbpl.mozilla.org/?tree=Fx-Team&rev=5cce921c930f https://hg.mozilla.org/integration/fx-team/rev/d698320dfba1
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d698320dfba1
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•