181037 2017-12-20 08:31:42 -0800 Refactor RenderMathMLMenclose to remove LayoutRect member modified during layout 2017-12-26 00:02:48 -0800 1 1 1 Unclassified WebKit MathML WebKit Nightly Build Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=161126 P2 Normal --- 180348 1 fred.wang webkit-unassigned alex jfernandez lambda rego oldest_to_newest 1383645 0 fred.wang 2017-12-20 08:31:42 -0800 fred@igalia-fwang:~/webkit/WebKit$ grep m_ Source/WebCore/rendering/mathml/RenderMathMLMenclose.h LayoutRect m_contentRect; This is similar to bug 180151. We can probably refactor the code so that we don't need m_contentRect. The RenderMathMLRow::getContentBoundingBox function introduced in bug 180348 should help to do it. 1384697 1 lambda 2017-12-26 00:02:48 -0800 Is this really necessary? If we decide to introduce a const method to replace m_contentRect, then we need to recompute the content bounding box every time we paint, which seems quite costly to me. How about caching the content bounding box in RenderMathMLRow so that we do not need to cache it twice? That seems more manageable.