Closed
Bug 1512256
Opened 6 years ago
Closed 6 years ago
Move FragmentOrElement::mAttrs to Element
Categories
(Core :: DOM: Core & HTML, enhancement)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla65
| Tracking | Status | |
|---|---|---|
| firefox65 | --- | fixed |
People
(Reporter: smaug, Assigned: smaug)
Details
Attachments
(1 file)
|
9.71 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
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)
|
||
Updated•6 years ago
|
Attachment #9029652 -
Flags: review?(ehsan) → review+
Pushed by opettay@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/10320b5096c2 Move FragmentOrElement::mAttrs to Element, r=ehsan
|
||
Comment 2•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/10320b5096c2
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•