Move FragmentOrElement::mAttrs to Element

RESOLVED FIXED in Firefox 65

Status

()

enhancement
RESOLVED FIXED
6 months ago
5 months ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

unspecified
mozilla65
Points:
---

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 months ago
This should pack Elements in memory a bit more efficiently on some 32 bit systems. (but I still haven't figured out how to get more bits for Text nodes)

Decided to keep FragmentOrElement's Traverse as it is. It already deals with Elements in various ways and cleaning that up would be a separate bug.


https://treeherder.mozilla.org/#/jobs?repo=try&revision=509cc94e38e2bc0165fa6bb946cb8bd509c3738d
Attachment #9029652 - Flags: review?(ehsan)
Attachment #9029652 - Flags: review?(ehsan) → review+

Comment 1

5 months ago
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/10320b5096c2
Move FragmentOrElement::mAttrs to Element, r=ehsan

Comment 2

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/10320b5096c2
Status: NEW → RESOLVED
Last Resolved: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.