Closed
Bug 1437157
Opened 7 years ago
Closed 6 years ago
Increase coverage of Stack Labels
Categories
(Core :: Gecko Profiler, enhancement, P1)
Core
Gecko Profiler
Tracking
()
RESOLVED
FIXED
People
(Reporter: gregtatum, Assigned: canova)
References
(Blocks 1 open bug)
Details
Now that PseudoStacks are cheaper, we should instrument more of Gecko to provide better annotations. This information will then more useful to expose to perf.html.
❏ Evaluate Script (AutoEntryScript)
❏ Regex matching
❏ Regex compilation
❏ Script load (entry point to JS execution)
❏ Baseline compilation
❏ Ion compilation
❏ CSS parsing
❏ Selector matching
❏ Style flush
❏ Frame construction
❏ Dll loading
❏ Network
❏ All things that call into JS
❏ DOM event
❏ Initial layout
❏ JS parsing
❏ Load JSM
❏ DestroyWindowEvent
❏ Incremental marking
❏ ForgetSkippable
❏ CC
❏ GC
❏ WebIDL call
❏ Event loop / idle
❏ HTML parsing
❏ Layout Flush
❏ Refresh Tick
Reporter | ||
Updated•7 years ago
|
Priority: -- → P1
Reporter | ||
Updated•7 years ago
|
Summary: Increase coverage of PseudoStack entires → Increase coverage of PseudoStack entries
Reporter | ||
Updated•6 years ago
|
Summary: Increase coverage of PseudoStack entries → Increase coverage of Stack Labels
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → canaltinova
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•6 years ago
|
||
So, after the bug 1478393, it appears everything is finished on my list. The other label frames on that list were already added.
Markus, can we close this now?
Flags: needinfo?(mstange)
Comment 2•6 years ago
|
||
I've filed bug 1480496 and bug 1480499 for two more that I found in my notes, but after those we can close this.s
Flags: needinfo?(mstange)
Assignee | ||
Comment 3•6 years ago
|
||
I'm closing this since other bugs are fixed now. Thanks Markus!
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Comment 4•6 years ago
|
||
Maybe bug 1442665 is in the scope of this bug too ? What do you think Nazim and Markus ?
Assignee | ||
Comment 5•6 years ago
|
||
It looks like it's not in the list above but yes, we can also work on this bug as a next step.
You need to log in
before you can comment on or make changes to this bug.
Description
•