Closed
Bug 1340709
Opened 7 years ago
Closed 7 years ago
Tweak the profiler markers for the different refresh driver phases
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: mstange, Assigned: mstange)
References
(Blocks 1 open bug)
Details
Attachments
(4 files)
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 | ||
Updated•7 years ago
|
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 6•7 years ago
|
||
mozreview-review |
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 7•7 years ago
|
||
mozreview-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 8•7 years ago
|
||
mozreview-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 9•7 years ago
|
||
mozreview-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+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 14•7 years ago
|
||
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
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/19da94c09bfd https://hg.mozilla.org/mozilla-central/rev/207d018b3dff https://hg.mozilla.org/mozilla-central/rev/f8f4716a3529 https://hg.mozilla.org/mozilla-central/rev/197342afdeff
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•