only call NotifyContentfulPaintForRootContentDocument for root content documents
Categories
(Core :: Performance, defect)
Tracking
()
People
(Reporter: tnikkel, Unassigned)
Details
Attachments
(1 obsolete file)
https://hg.mozilla.org/integration/autoland/rev/47629f89cb6a
changed it so that mFirstContentfulPaintTransactionId was set to something for non-root content documents. In nsPresContext::NotifyDidPaintForSubtree we use the existence of mFirstContentfulPaintTransactionId to determine to call NotifyContentfulPaintForRootContentDocument.
If the intent is to call NotifyContentfulPaintForRootContentDocument for all documents we should change the name, but as it is there are a couple other similarly named functions that do only get called for root content documents so bringing these all to be the same would be good.
Reporter | ||
Comment 1•3 years ago
|
||
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Reporter | ||
Comment 2•3 years ago
|
||
Based on the comments in https://phabricator.services.mozilla.com/D124273 I may have mis-understood the problem here. Un-assigning.
Comment 3•3 years ago
|
||
Set release status flags based on info from the regressing bug 1518999
Comment 5•3 years ago
|
||
Yeah, I think that's what we'd like to do.
Updated•3 years ago
|
Description
•