Closed
Bug 107100
Opened 23 years ago
Closed 13 years ago
move attribute mapping out of content nodes
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
WONTFIX
Future
People
(Reporter: dbaron, Assigned: dbaron)
References
Details
We should move the attribute mapping code out of the content nodes for the
following reasons:
* It would be easier to maintain if it's in one place, in the style system
(near nsHTMLStyleSheet.cpp and nsHTMLMappedAttributes.cpp).
* The attribute mapping code doesn't logically belong in content -- content
shouldn't know about the style system, except for exposing a few DOM methods.
* If we don't recombine layout and content, it's the main obstacle to moving
all style system code back into the layout library.
* It gives the DOM folks one more excuse to keep all the separate content node
implementations with the forwarding, etc.
The tag names could be handled using a perfect hash.
Seehttp://lxr.mozilla.org/classic/source/lib/libparse/
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → Future
Assignee | ||
Comment 1•23 years ago
|
||
Comment 2•23 years ago
|
||
If you're going to rewrite this, could you make the apis non-html specific, so
that SVG can take advantage of this without having to copy code arround?
Assignee | ||
Comment 3•23 years ago
|
||
There are also similar tools in
http://lxr.mozilla.org/seamonkey/source/layout/tools/
Assignee | ||
Comment 4•23 years ago
|
||
See also the directory structure proposed in bug 107101.
Comment 5•23 years ago
|
||
Yeah, the hash would have to be keyed on the namespace as well as the tagname.
Assignee | ||
Updated•18 years ago
|
QA Contact: ian → style-system
Assignee | ||
Comment 6•13 years ago
|
||
I think this was based on a false notion of modularity.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•