Open Bug 811927 Opened 11 years ago Updated 6 months ago

Don't use a temporary surface for inactive opacity surfaces if we don't need to


(Core :: Layout, defect)




Performance Impact fixed
a11y-review fixed
Webcompat Priority fixed
Tracking Status
relnote-firefox --- fixed
thunderbird_esr91 fixed fixed
thunderbird_esr102 fixed fixed
firefox-esr102 fixed fixed
firefox111 fixed fixed
firefox112 fixed fixed
firefox113 fixed fixed


(Reporter: mattwoodrow, Unassigned)



(Whiteboard: [Snappy:p1][leave open])


(1 file, 1 obsolete file)

Attached patch Skip PushGroup if we can (obsolete) — Splinter Review
We do this a lot when painting gmail, this *should* help with performance there a fair bit.
Attachment #681724 - Flags: review?(roc)

I realise that there are a lot more display item types that could probably return false here, we should do that as a followup.
Comment on attachment 681724 [details] [diff] [review]
Skip PushGroup if we can

Review of attachment 681724 [details] [diff] [review]:

I think this prevents opacity from being applied at all.

I suggest turning these layers into ImageLayers.
This is simple and worth having.

I have a WIP patch that fixes the other patch in this bug. It works, but needs a lot of tidying up.
Attachment #682270 - Flags: review?(roc)
Attachment #681724 - Attachment is obsolete: true
Attachment #681724 - Flags: review?(roc)
Whiteboard: [Snappy:p1]
Whiteboard: [Snappy:p1] → [Snappy:p1][leave open]
Depends on: 816692
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.