Created attachment 47631 [details] MathML CSS patch This patch removed the unnecessary line-height settings and updates the existing tests.
Comment on attachment 47631 [details] MathML CSS patch Your ChangeLog does not explain why it's unecessary. Or how it positively effects the layout test results?
(In reply to comment #1) > (From update of attachment 47631 [details]) > Your ChangeLog does not explain why it's unecessary. Or how it positively > effects the layout test results? At this point, this updated CSS or the previous one will not make much of a difference in terms of the line-height setting. The line-height settings are from experiments from earlier code. The negatively affect other rendering objects that are in other patches. They only make minor adjustments to the current tests that are almost imperceptible. This is basically a cleanup of the CSS in preparation for more rendering objects and tests in the future.
Comment on attachment 47631 [details] MathML CSS patch OK. Should say more of that in the ChangeLog in the future. :) But this is OK. Thanks for the explanation.
Comment on attachment 47631 [details] MathML CSS patch Clearing flags on attachment: 47631 Committed r54029: <http://trac.webkit.org/changeset/54029>
All reviewed patches have been landed. Closing bug.