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)

defect

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/
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → Future
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?
See also the directory structure proposed in bug 107101.
Blocks: 114713
Yeah, the hash would have to be keyed on the namespace as well as the tagname.
Depends on: 127269
QA Contact: ian → style-system
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.