[Webkit-unassigned] [Bug 155879] Use LayoutUnit values for MathML ascent

[bugzilla-daemon at webkit.org]

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

Frédéric Wang (:fredw) <fred.wang at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |distler at golem.ph.utexas.edu
         Depends on|85730                       |161136, 161127

--- Comment #4 from Frédéric Wang (:fredw) <fred.wang at free.fr> ---
I'm not exactly sure why firstLineBaseline() must return an (Optional) int for MathML (besides the fact that MathML renderer classes used to derive from flexboxes). I think we want to use LayoutUnit as much as possible.

I'm making this depends on bug 161136 and bug 161127 which allow to implement firstLineBaseline() without having to store LayoutUnit members and keep them updated. In general, it seems that performing a simple computation is always better than having to use a (possible not up-to-date class member).

I'm cc'int Jacques Distler as he reported an issue with the alignment of attachment 286836 that I suspect is due to this bug (but I was not able to reproduce it).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160824/13058f72/attachment-0001.html>