RESOLVED FIXED115786
Invisible Operators should not add space 
https://bugs.webkit.org/show_bug.cgi?id=115786
Summary Invisible Operators should not add space
Frédéric Wang Nélar
Reported 2013-05-08 02:49:50 PDT
Created attachment 201047 [details] testcase In the attached test case (from the MathML spec), you see that invisible operators add space to the mathematical formulas. However, they should have width=0 and per the Operator dictionary (http://www.w3.org/TR/MathML/appendixc.html) no space around them.
Attachments
testcase (894 bytes, text/html)
2013-05-08 02:49 PDT, Frédéric Wang Nélar 		no flags
Details
Patch (7.05 KB, patch)
2014-02-26 01:46 PST, Frédéric Wang Nélar 		no flags
DetailsFormatted DiffDiff
Patch (7.16 KB, patch)
2014-03-06 08:55 PST, Frédéric Wang Nélar 		no flags
DetailsFormatted DiffDiff
Patch + 115787 for testing (76.54 KB, patch)
2014-03-11 02:28 PDT, Frédéric Wang Nélar 		no flags
DetailsFormatted DiffDiff
Patch (7.12 KB, patch)
2014-03-11 03:22 PDT, Frédéric Wang Nélar 		cfleizach: review+
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Frédéric Wang Nélar
Comment 1 2013-05-15 05:43:28 PDT
A Chromium bug was reported last January: http://code.google.com/p/chromium/issues/detail?id=169753
Frédéric Wang Nélar
Comment 2 2014-02-26 01:46:22 PST
Created attachment 225243 [details] Patch
Frédéric Wang Nélar
Comment 3 2014-03-06 08:55:43 PST
Created attachment 225996 [details] Patch Just refreshing the patch.
chris fleizach
Comment 4 2014-03-06 10:28:11 PST
Comment on attachment 225996 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=225996&action=review is this ready for r? > Source/WebCore/rendering/mathml/RenderMathMLOperator.h:91 > + bool isInvisibleOperator() { return 0x2061 <= m_operator && m_operator <= 0x2064; } this can be const
Frédéric Wang Nélar
Comment 5 2014-03-06 11:28:55 PST
(In reply to comment #4) > (From update of attachment 225996 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=225996&action=review > > is this ready for r? I don't think the patch will change much, but this applies on top of the patch of bug 115787 which itself applies on top of bug 124838... Did you find anything about how to make the accessibility test pass after the patch for bug 124838?
Frédéric Wang Nélar
Comment 6 2014-03-11 02:28:51 PDT
Created attachment 226402 [details] Patch + 115787 for testing
Frédéric Wang Nélar
Comment 7 2014-03-11 03:22:21 PDT
Created attachment 226411 [details] Patch This applies on top of 115787.
chris fleizach
Comment 8 2014-03-11 09:19:42 PDT
Comment on attachment 226411 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=226411&action=review > Source/WebCore/rendering/mathml/RenderMathMLOperator.cpp:1306 > + // In some fonts, glyphs for invisible operators have nonzero width. Consequently, we substract that width here to avoid wide gaps. substract -> subtract
Frédéric Wang Nélar
Comment 9 2014-03-12 02:50:37 PDT
Committed r165464: <http://trac.webkit.org/changeset/165464>
Frédéric Wang Nélar
Comment 10 2014-03-12 03:26:04 PDT
Interestingly, <p>_&#x2062;_</p> does not produce space on Gecko but it does on WebKit. According to Khaled Hosny, HarfBuzz handles these characters specifically: https://bugzilla.mozilla.org/show_bug.cgi?id=522393#c43
Note You need to log in before you can comment on or make changes to this bug.