Closed
Bug 925990
Opened 11 years ago
Closed 11 years ago
Profile has pink text on top of it
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 27
People
(Reporter: heroldtom, Assigned: past)
References
Details
Attachments
(2 files, 1 obsolete file)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
anton
:
review+
|
Details | Diff | Splinter Review |
I am using nightly 27.0a1 from 23/10/11. When I record a profile the "graph" gets overdrawn with Pink text saying: [Object Object]. Anton checked out and rules out addons/extensions and probably some other stuff. (ask him personally) See the attached screenshot.
Assignee | ||
Comment 1•11 years ago
|
||
I'm going to look into this.
Assignee: nobody → past
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•11 years ago
|
||
This is caused by bug 867757 which changed markers to be objects instead of strings, and bug 918825 has increased the number of such markers. I don't think displaying the markers in the graph makes sense any longer, especially when you zoom in a particular area of the graph, where the text just looks stretched and ugly. What I'd like to see is these events to appear as vertical bars with different colors per category/type and display a legend somewhere that explains what each color stands for. Maybe even show tooltips when hovering over these bars. I'm not sure how far the profiler UI rewrite has progressed though, and whether it makes sense to do this in Cleopatra.
Comment 3•11 years ago
|
||
We've changed how markers are displayed in our repo to be much nicer. You should by default hide non text markers.
Assignee | ||
Comment 5•11 years ago
|
||
Benoit, you mean like this? Which markers are going to be plain strings? I tried to just use marker.name, but the display was too cluttered even with the right labels and zooming in needs more work as it still results in ugly text.
Attachment #822307 -
Flags: review?(bgirard)
Comment 6•11 years ago
|
||
Comment on attachment 822307 [details] [diff] [review] Ignore non-string markers Going forward all markers will an object. Maybe you want to do it if the type is tracing. http://mxr.mozilla.org/mozilla-central/source/tools/profiler/ProfilerMarkers.cpp#71
Attachment #822307 -
Flags: review?(bgirard) → review-
Assignee | ||
Comment 7•11 years ago
|
||
OK, in that case I think the best short-term solution is to not display markers altogether. Displaying them properly can be done after the UI rewrite.
Attachment #822378 -
Flags: review?(anton)
Assignee | ||
Updated•11 years ago
|
Attachment #822307 -
Attachment is obsolete: true
Updated•11 years ago
|
Attachment #822378 -
Flags: review?(anton) → review+
Assignee | ||
Comment 8•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/60c2fea83c0f
Whiteboard: [fixed-in-fx-team]
Comment 9•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/60c2fea83c0f
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 27
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•