NEW 181037
Refactor RenderMathMLMenclose to remove LayoutRect member modified during layout 
https://bugs.webkit.org/show_bug.cgi?id=181037
Summary Refactor RenderMathMLMenclose to remove LayoutRect member modified during layout
Frédéric Wang (:fredw)
Reported 2017-12-20 08:31:42 PST
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.
Attachments
Add attachment proposed patch, testcase, etc.
Minsheng Liu
Comment 1 2017-12-26 00:02:48 PST
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.
Note You need to log in before you can comment on or make changes to this bug.