Open
Bug 515349
Opened 15 years ago
Updated 2 years ago
Over 1000 PresShell::FlushPendingNotifications during blank page startup
Categories
(Firefox :: General, defect)
Tracking
()
NEW
People
(Reporter: joelr, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: perf, Whiteboard: [ts])
Attachments
(2 files)
There are over 1000 PresShell::FlushPendingNotifications during blank page startup. I don't know whether all of them do work, only 9 trigger XUL document reflows in ProcessReflowCommands. The method itself is quite branchy, with lots of ifs. I also wonder if some of the flushes can be optimized away. It could be that the same things are flushed over and over again and I'm curious if flushed could be queued up and duplicate or unnecessary flushes eliminated.
Reporter | ||
Comment 1•15 years ago
|
||
To be more precise, there are 1109 flushes of which 9 trigger XUL document reflows with root count greater than 0. Boris suggested to bin this by flush type, fully expecting most of these to be style flushes, assuming there will only be three bins (style/layout/paint) and that the paint bin will be empty. I'm working on this.
Reporter | ||
Comment 2•15 years ago
|
||
Call sites of all the blank page startup flushes.
Reporter | ||
Comment 3•15 years ago
|
||
Reporter | ||
Comment 4•15 years ago
|
||
Boris, this is what you asked for, pending flushes by type. ./d firefox-bin flush-pending-by-type.d dtrace -Zqw -x dynvarsize=64m -x evaltime=preinit -p 13540 -s flush-pending-by-type.d -s sigcont.d ---------------------- type 4, elapsed: 0.1458990ms type 3, elapsed: 3.7413888ms type 5, elapsed: 18.2747276ms type 4, cpu : 0.1288225ms type 3, cpu : 3.5871132ms type 5, cpu : 6.2415977ms type 5, count : 27 times type 4, count : 35 times type 3, count : 1046 times
Reporter | ||
Comment 5•15 years ago
|
||
Flush_Style = 3, /* As above, plus flush style reresolution */ Flush_Frames = Flush_Style, Flush_Layout = 4, /* As above, plus flush reflow */ Flush_Display = 5 /* As above, plus flush painting */
Reporter | ||
Comment 6•15 years ago
|
||
The above is wrong, my apologies. This is from the trunk: Flush_Style = 3, /* As above, plus flush style reresolution */ Flush_Frames = Flush_Style, Flush_InterruptibleLayout = 4, /* As above, plus flush reflow, but allow it to be interrupted (so an incomplete layout may result) */ Flush_Layout = 5, /* As above, but layout must run to completion */ Flush_Display = 6 /* As above, plus flush painting */
Reporter | ||
Comment 7•15 years ago
|
||
bz: joelr: ok, so almost all Flush_Frames/Flush_Style flushes bz: joelr: so we're seeing one style flush per menuitem bz: joelr: (all of them no-ops, of course) bz: er... bz: correction joelr: bz: how do you tell these are menu items? bz: it's not one flush per menuitem bz: It's N per menuitem bz: where N is the number of parents bz: XUL`PresShell::FlushPendingNotifications(mozFlushType)+0x15d bz: XUL`nsComputedDOMStyle::GetPropertyCSSValue(nsAString_internal const&, nsIDOMCSSValue**)+0x20b bz: XUL`nsMenuItemIconX::GetIconURI(nsIURI**)+0x29c bz: XUL`nsMenuItemIconX::SetupIcon()+0x52 bz: XUL`nsMenuX::LoadMenuItem(nsIContent*)+0x131 bz: XUL`nsMenuX::MenuConstruct()+0xcd bz: is the stack bz: so I can tell they're menuitems by looking at those class names. bz: I think almost all of these are the menu stuff bz: Might be worth seeing whether we can make that not flush just to cut down on the noise bz: because there are no pending style changes bz: joelr: in any case, there are about 60 calls here that could trigger reflow, right? bz: joelr: the rest is just restyling stuff joelr: bz: i don't know how to tell if the calls "could" trigger reflow bz: joelr: might be worth seeing whether we can fix that menu thing, though bz: joelr: by looking at the flush type bz: joelr: and comparing to the mozFlushType enum values
Reporter | ||
Updated•15 years ago
|
Whiteboard: [ts]
Comment 8•15 years ago
|
||
It's probably worth filing a separate bug on the MenuConstruct() setup, at least if MenuConstruct() takes an appreciable amount of time.
Comment 9•2 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.
Assignee: joelr → nobody
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•