[Webkit-unassigned] [Bug 137330] RenderMathMLUnderOver adds spacing to the child operator indefinitely when resizing the window

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=137330


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #239254|review?                     |review+
               Flag|                            |




--- Comment #6 from Darin Adler <darin at apple.com>  2014-10-03 23:30:40 PST ---
(From update of attachment 239254)
View in context: https://bugs.webkit.org/attachment.cgi?id=239254&action=review

> Source/WebCore/rendering/mathml/RenderMathMLUnderOver.cpp:77
> +    Vector<RenderMathMLOperator*> renderOperators;

Would be more efficient to put an inline capacity here so we don't do memory allocation unless we have an unusually large number of operators.

> Source/WebCore/rendering/mathml/RenderMathMLUnderOver.cpp:97
> +    for (auto& renderOperator : renderOperators) {
> +        renderOperator->stretchTo(stretchWidth);
>      }

No braces on a 1-line for statement body in WebKit coding style.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.