Closed Bug 861863 Opened 12 years ago Closed 8 years ago

Modify Cleopatra to support multi-thread profiling

Categories

(Core :: Gecko Profiler, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: BenWa, Unassigned)

References

Details

Attachments

(1 file)

I did some rough changes to support multiple timeline but they have many bugs remaining. What I'd like to achieve: * Support zooming correctly * Sync up the thread timeline * The diagnostic bar should select the proper thread * Display markers in a more useful way Here's a sample profile collected by bent that we display miserably. I think this is a good target to improve in this bug: http://people.mozilla.com/~bgirard/cleopatra/#report=8987a832d7e5f7f224388eccf5b64a8e70919d6a
cc'ing anton. We'll do what we can to help out.
Comment 0 is mostly everything we need. Here are a few others that aren't entirely required but would make life much better: * Allow threads to be hidden from UI * Maybe disable recording on certain threads (performance on phone is very bad with all enabled) * Make horizontal scrollbar always visible (currently you have to scroll to bottom of the tree view to see it)
Blocks: 758697
Opened a PR to get feedback: https://github.com/bgirard/cleopatra/pull/1
Assignee: nobody → anton
Status: NEW → ASSIGNED
Attached image Screenshot (WIP) (deleted) —
I've accepted Anton changes and have been adding a proper timeline and time markers. I plan on doing a few tweaks for improving the display of markers, diagnostics items and tertiary items before deploying it. Shouldn't be to long. In the mean time here's the current WIP, feel free to give us early feedback.
Assignee: anton → nobody
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: