This is a small follow-up of the RenderMathMLRow/RenderMathMLUnderOver/RenderMathMLFraction refactoring. Since these MathML renderers can only contain other MathML renderers, we can just considerer RenderBox children and prevent unnecessary casts. Similarly, when the two arguments of std::max are LayoutUnit's, we do not need to specialize to std::max<LayoutUnit>.
Created attachment 279480 [details] Patch
Comment on attachment 279480 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=279480&action=review > Source/WebCore/rendering/mathml/RenderMathMLUnderOver.cpp:58 > + RenderBox* base = firstChildBox(); Typically in new code, we would use auto* here instead of writing RenderBox*. > Source/WebCore/rendering/mathml/RenderMathMLUnderOver.cpp:78 > + for (RenderBox* child = firstChildBox(); child; child = child->nextSiblingBox()) { Ditto.
https://trac.webkit.org/changeset/201418