Tweak the profiler markers for the different refresh driver phases

RESOLVED FIXED in Firefox 54

Status

()

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mstange, Assigned: mstange)

Tracking

(Blocks 1 bug)

Trunk
mozilla54
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 fixed)

Details

Attachments

(4 attachments)

At the moment we have:
 - "RD" for the whole refresh driver tick
 - "Scripts" for the requestAnimationFrame callbacks
 - "Styles"
 - "Reflow"
 - "DisplayList"
 - "Rasterize" from the beginning of rasterization to the middle of ForwardTransaction

I'm going to tweak these so that we have:
 - "RefreshDriverTick" for the whole refresh driver tick
 - "Scripts"
 - "Styles"
 - "FireScrollEvent"
 - "Reflow"
 - "DispatchSynthMouseMove"
 - "DisplayList"
 - "LayerBuilding"
 - "Rasterize"
 - "ForwardTransaction"
 - "NotifyDidPaint"
in the right places.
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Comment on attachment 8838766 [details]
Bug 1340709 - Add a profiler tracing marker for layer building.

https://reviewboard.mozilla.org/r/113568/#review115282
Attachment #8838766 - Flags: review?(matt.woodrow) → review+
Comment on attachment 8838767 [details]
Bug 1340709 - Add profiler tracing markers for more phases of the refresh tick.

https://reviewboard.mozilla.org/r/113570/#review115284
Attachment #8838767 - Flags: review?(matt.woodrow) → review+
Comment on attachment 8838768 [details]
Bug 1340709 - Don't include ForwardTransaction time in the Rasterize tracing marker.

https://reviewboard.mozilla.org/r/113572/#review115286
Attachment #8838768 - Flags: review?(matt.woodrow) → review+
Comment on attachment 8838769 [details]
Bug 1340709 - Rename the profiler tracing marker for the refresh driver tick from RD to RefreshDriverTick.

https://reviewboard.mozilla.org/r/113574/#review115288
Attachment #8838769 - Flags: review?(matt.woodrow) → review+
Pushed by mstange@themasta.com:
https://hg.mozilla.org/integration/autoland/rev/19da94c09bfd
Add a profiler tracing marker for layer building. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/207d018b3dff
Add profiler tracing markers for more phases of the refresh tick. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/f8f4716a3529
Don't include ForwardTransaction time in the Rasterize tracing marker. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/197342afdeff
Rename the profiler tracing marker for the refresh driver tick from RD to RefreshDriverTick. r=mattwoodrow
You need to log in before you can comment on or make changes to this bug.