Closed
Bug 879007
Opened 11 years ago
Closed 11 years ago
Prototype for the new profiler UI
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: anton, Assigned: anton)
Details
Attachments
(1 file)
18.00 KB,
patch
|
dcamp
:
feedback+
|
Details | Diff | Splinter Review |
First prototype with a tree of calls and a chart.
Assignee | ||
Comment 1•11 years ago
|
||
Make the profile visualization interface interchangeable so that we could put new UI under the flag and land it sooner than later. First step was to move ProfileUI into a separate file (I also renamed it to Cleopatra). And since I was moving stuff I changed it to use the new Addon SDK. And it even works! Dave, did I miss anything? (I assume you know best since you were working on porting other modules)
Attachment #765068 -
Flags: feedback?(dcamp)
Comment 2•11 years ago
|
||
Comment on attachment 765068 [details] [diff] [review] Prereq refactoring Any reason not to convert ProfilerPanel too?
Assignee | ||
Comment 3•11 years ago
|
||
I will later. For now I'm splitting patches to make reviews easier and land them earlier.
Comment 4•11 years ago
|
||
Comment on attachment 765068 [details] [diff] [review] Prereq refactoring Sounds good, f+ then.
Attachment #765068 -
Flags: feedback?(dcamp) → feedback+
Assignee | ||
Comment 5•11 years ago
|
||
Waiting on bug 900918 to be fixed: otherwise ProfileStart is failing. (See https://bugzilla.mozilla.org/show_bug.cgi?id=900918#c5 for more info)
Assignee | ||
Comment 6•11 years ago
|
||
The prototype is here: https://github.com/antonkovalyov/moz-profile-histogram Marking this as RESOLVED FIXED since I'm about to upload a patch (see bug 879008) where I put that prototype into Firefox.
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•