Closed Bug 822110 Opened 11 years ago Closed 11 years ago

filter.png, noise.png and showall.png in browser/devtools/profiler/cleopatra/images are empty


(DevTools :: Performance Tools (Profiler/Timeline), defect)

Not set


(Not tracked)

Firefox 20


(Reporter: rcampbell, Assigned: anton)



(Whiteboard: [profiler])


(1 file, 1 obsolete file)

Empty files landed in profiler/cleopatra. We should remove them if unused.
Whiteboard: [profiler]
Assignee: nobody → anton
Attachment #693132 - Flags: review?(rcampbell)
Attachment #693132 - Flags: review?(bgirard)
Depends on: 795268
noise.png is also empty, and referred to in ui.css.
It should be removed, as well as the corr. part in the background-image statement, as it has not effect at all being empty).

Furthermore the -webkit- and -moz- lines should be removed from the cleopatra/*.css files.
If you look at the attached patch you'll see that noise.png is not empty anymore. I'll remove prefixed CSS properties.
Comment on attachment 693132 [details] [diff] [review]
Updated Cleo light and removed unused image files.

+@-webkit-keyframes darken {
+  from {

ugh. ok.
Attachment #693132 - Flags: review?(rcampbell) → review+
Comment on attachment 693132 [details] [diff] [review]
Updated Cleo light and removed unused image files.

should also add a non-prefixed version of the @keyframes rules for fx 16+.
can we ditch the -webkit and -moz prefixes entirely for this? Since cleo-light exists solely for inclusion in the browser, there's not much reason to keep them.
Component: Developer Tools → Developer Tools: Profiler
OS: Mac OS X → All
Hardware: x86 → All
Attachment #693132 - Flags: review?(bgirard) → review+
Ideally cleo-light would use the same JS/CSS files as the normal cleopatra except for having a flag set. However the extra prefixes causes a lot of error console noise :(. Note that this 'update' doesn't really pull in the newer changes from cleopatra. One notable change is being able to click on the histogram to select a sample.
Some -moz- properties are still there because shows that they are still prefixed in all version of Firefox.
Attachment #693132 - Attachment is obsolete: true
Attachment #697279 - Flags: review?(rcampbell)
Attachment #697279 - Flags: review?(bgirard)
Attachment #697279 - Flags: review?(rcampbell) → review+
we should submit these changes as a pull request to the cleo-light branch in github.
Attachment #697279 - Flags: review?(bgirard) → review+
Whiteboard: [profiler] → [profiler][land-in-fx-team]
Whiteboard: [profiler][land-in-fx-team] → [profiler][fixed-in-fx-team]
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [profiler][fixed-in-fx-team] → [profiler]
Target Milestone: --- → Firefox 20
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.