Closed
Bug 1483964
Opened 6 years ago
Closed 6 years ago
Manually inline class and ID getters for rust.
Categories
(Core :: CSS Parsing and Computation, enhancement, P2)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(2 files)
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.
|
Assignee | |
Comment 1•6 years ago
|
||
They're the same, and they aren't complex enough to be an AttrArray
implementation detail I think.
|
|
Assignee | |
Comment 2•6 years ago
|
||
Somewhat ugly but hopefully not too much.
Somehow it ends up removing more lines than adding.
|
||
Comment 3•6 years ago
|
||
Comment on attachment 9001713 [details]
Unify InternalAttr and ServoAttrSnapshot.
Olli Pettay [:smaug] has approved the revision.
Attachment #9001713 -
Flags: review+
|
||
Comment 4•6 years ago
|
||
Comment on attachment 9001713 [details]
Unify InternalAttr and ServoAttrSnapshot.
Xidorn Quan [:xidorn] UTC+10 has approved the revision.
Attachment #9001713 -
Flags: review+
|
|
||
Comment 5•6 years ago
|
||
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
|
||
Comment 7•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/a7a2da45f8f7
https://hg.mozilla.org/mozilla-central/rev/62d4efbe32d7
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•