Currently the class hierarchy looks like: /* <mi> identifier - italics *sometimes*, i.e., context-sensitive */ nsMathMLmiFrame ------------+ | /* <mtext> vanilla text */ | nsMathMLmtextFrame ---------+--- nsMathMLContainerFrame --- nsHTMLContainerFrame | /* <ms> string which needs to be rendered with generated "quotes" */ nsMathMLmsFrame ------------+ The custom behaviors of each tag is handled by functions in these files. I would like to merge these files to cut the C++, and use the good old tag test to call the relevant functions. These are the MathML tags that have child nsTextFrames. Since text frames are non-MathML, they often require special handling. Merging will help to centralize the special handling. In particualar, the support of the |mathvariant| attribute needs special transformation in the text. Bug 114365 comment 2 describes the full scope of the issues needed to support this mathvariant attribute. This bug is a sub-task in the chain.

Welcoming reviews at this stage.

bz/roc, care to review? It is essentially a shuffling of functions. Will be good to land this early in the milestone ot let it bake a bit.

bz said that he is about to catch a plane. pinging kin...

Comment on attachment 95142 [details] [diff] [review] iteration to clean those old .get() in the code being moved by the patch r=roc+moz usual caveats apply...

Comment on attachment 95142 [details] [diff] [review] iteration to clean those old .get() in the code being moved by the patch sr=kin@netscape.com I'm guessing mathml isn't built on Mac since there aren't any mac xml project diffs, and no project xml files in the mathml dir?

oops, it is now built by default, after a loooong time... will take care of the xml project stuff.

Comment on attachment 95635 [details] [diff] [review] update to sync with recent changes on the trunk and include macbuild diffs migrating r/sr

Checked in.