Cache nsIFrame::BackfaceIsHidden() and nsIFrame::Combines3DTransformWithAncestors() in display items

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P3
normal
RESOLVED FIXED
6 months ago
2 months ago

People

(Reporter: miko, Assigned: miko)

Tracking

unspecified
mozilla67
Points:
---

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

6 months ago
Currently FLB/DL building caches In3DContextAndBackfaceIsHidden(), which is slower than BackfaceIsHidden(). I think we can use the container frame information to figure out if we are inside 3D transform and use the simple version.

This also allows us to cache the value for WebRender.
Assignee

Updated

6 months ago
Priority: -- → P3
Assignee

Updated

2 months ago
Summary: Cache BackfaceIsHidden() in display items instead of In3DContextAndBackfaceIsHidden() → Cache nsIFrame::BackfaceIsHidden() and nsIFrame::Combines3DTransformWithAncestors() in display items
Attachment #9027916 - Attachment description: Bug 1510286 - Cache BackfaceIsHidden() in display items instead of In3DContextAndBackfaceIsHidden() → Bug 1510286 - Cache nsIFrame::BackfaceIsHidden() and nsIFrame::Combines3DTransformWithAncestors() in display items

Comment 2

2 months ago
Pushed by mikokm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/9c7c3ca16387
Cache nsIFrame::BackfaceIsHidden() and nsIFrame::Combines3DTransformWithAncestors() in display items r=mattwoodrow

Comment 3

2 months ago
bugherder
Status: ASSIGNED → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.