Closed
Bug 1418836
Opened 8 years ago
Closed 8 years ago
No restyle markers inside refresh ticks anymore
Categories
(Core :: Gecko Profiler, defect)
Core
Gecko Profiler
Tracking
()
RESOLVED
FIXED
mozilla59
| Tracking | Status | |
|---|---|---|
| firefox-esr52 | --- | unaffected |
| firefox57 | --- | unaffected |
| firefox58 | --- | unaffected |
| firefox59 | --- | fixed |
People
(Reporter: mstange, Assigned: mstange)
References
Details
(Keywords: regression)
Attachments
(1 file)
PresShell::DoFlushPendingNotifications calls ProcessPendingRestyles in two different places. In bug 1350930 I only instrumented one of them. As a consequence, some restyle time is now unaccounted for when you only look at the markers.
| Comment hidden (mozreview-request) |
|
||
Comment 2•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8929892 [details]
Bug 1418836 - Add restyle markers in a place that I missed in bug 1350930.
https://reviewboard.mozilla.org/r/201062/#review206224
Attachment #8929892 -
Flags: review?(bzbarsky) → review+
Pushed by mstange@themasta.com:
https://hg.mozilla.org/integration/autoland/rev/45a167ac79a5
Add restyle markers in a place that I missed in bug 1350930. r=bz
|
|
||
Comment 4•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
|
||
Updated•8 years ago
|
status-firefox57:
--- → unaffected
status-firefox58:
--- → unaffected
status-firefox-esr52:
--- → unaffected
You need to log in
before you can comment on or make changes to this bug.
Description
•