Manually inline class and ID getters for rust.

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P2
normal
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: emilio, Assigned: emilio)

Tracking

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(2 attachments)

This speeds up bug 1422522 considerably, from 1800ms to 1450ms on my machine. It also improves bug 1468739.

We can do this without duplicating too much magic from AttrArray after bug 1482866.
They're the same, and they aren't complex enough to be an AttrArray
implementation detail I think.
Somewhat ugly but hopefully not too much.

Somehow it ends up removing more lines than adding.
Comment on attachment 9001713 [details]
Unify InternalAttr and ServoAttrSnapshot.

Olli Pettay [:smaug] has approved the revision.
Attachment #9001713 - Flags: review+
Comment on attachment 9001713 [details]
Unify InternalAttr and ServoAttrSnapshot.

Xidorn Quan [:xidorn] UTC+10 has approved the revision.
Attachment #9001713 - Flags: review+
Comment on attachment 9001714 [details]
Manually inline class and ID getters.

Xidorn Quan [:xidorn] UTC+10 has approved the revision.
Attachment #9001714 - Flags: review+
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a7a2da45f8f7
Unify InternalAttr and ServoAttrSnapshot. r=smaug,xidorn
https://hg.mozilla.org/integration/mozilla-inbound/rev/62d4efbe32d7
Manually inline class and ID getters. r=xidorn
https://hg.mozilla.org/mozilla-central/rev/a7a2da45f8f7
https://hg.mozilla.org/mozilla-central/rev/62d4efbe32d7
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.