[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





--- Comment #10 from Said Abou-Hallawa <sabouhallawa at apple.com>  2014-10-07 19:50:08 PST ---
(In reply to comment #7)
> (From update of attachment 239254 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=239254&action=review
> 
> > LayoutTests/mathml/presentation/munderover-window-resize.html:5
> > +  <math xmlns='http://www.w3.org/1998/Math/MathML' display='block'>

Done.

> Nit: The attribute xmlns is unnecessary given that this is an HTML5 document (as specified by the DOCTYPE directive).
> 
> > LayoutTests/mathml/presentation/munderover-window-resize.html:28
> > +      </msub>         
> 
> Please remove the trailing whitespace on this line.

Done

> > LayoutTests/mathml/presentation/munderover-window-resize.html:87
> > +window.addEventListener("load", function(){
> 
> Nit: Is it necessary to register a Load event listener as opposed to inlining the contents of the function body into the <script>?

Done.I removed the event listener and now the javascript should run after loading the page.

> > LayoutTests/mathml/presentation/munderover-window-resize.html:90
> > +    window.resizeTo(250, window.outerHeight); 
> > +    window.resizeTo(width, window.outerHeight);
> 
> I take it we are guaranteed to perform a layout between these two calls?

Yes The layout runs because of the resizing or zooming.  I stepped in the x-code debugger and I made sure the layout runs for every time the view is zoomed or the window is resized.

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