123348 2013-10-25 11:04:59 -0700 [MathML] Layout invalid markup as an <mrow> 2024-07-22 16:38:18 -0700 1 1 1 Unclassified WebKit MathML 528+ (Nightly build) All All RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=254093 https://bugs.webkit.org/show_bug.cgi?id=179739 InRadar, WPTImpact P2 Normal --- 153991 276917 195797 1 bfulgham webkit-unassigned aestes ahmad.saleem792 alex commit-queue fred.wang mrobinson webkit-bug-importer oldest_to_newest 943663 0 bfulgham 2013-10-25 11:04:59 -0700 Currently, certain invalid MathML documents (such as a MathML document containing arbitrary HTML markup) is rendered improperly. According to the MathML specification, we should be providing better error information. From http://www.w3.org/TR/MathML3/chapter6.html), Section 6.4 Combining MathML and Other Formats: "In the standard schema, elements from other namespaces are not allowed, but attributes from other namespaces are permitted. MathML processors that encounter unknown XML markup should behave as follows: An attribute from a non-MathML namespace should be silently ignored. An element from a non-MathML namespace should be treated as an error, except in an annotation-xml element. If the element is a child of a presentation element, it should be handled as described in Section 3.3.5 Error Message <merror>. If the element is a child of a content element, it should be handled as described in Section 4.2.9 Error Markup <cerror>. For example, if the second child of an mfrac element is an unknown element, the fraction should be rendered with a denominator that indicates the error." 943736 1 mrobinson 2013-10-25 13:55:49 -0700 I looked at the torture tests in WebKitGTK+ and they seem fine. 943815 2 bfulgham 2013-10-25 16:32:06 -0700 See also http://www.whatwg.org/specs/web-apps/current-work/multipage/the-map-element.html#mathml: "User agents must act as if any MathML element whose contents does not match the element's content model was replaced, for the purposes of MathML layout and rendering, by an merror element in the MathML namespace containing some appropriate error message." 943954 3 215269 fred.wang 2013-10-27 05:25:03 -0700 Created attachment 215269 testcase 944120 4 bfulgham 2013-10-28 09:34:34 -0700 (In reply to comment #1) > I looked at the torture tests in WebKitGTK+ and they seem fine. I think this comment was intended for Bug 121728. :-) 1174394 5 fred.wang 2016-03-14 03:58:56 -0700 At the moment, the choice in the in-progress MathML refactoring is just to not display anything for invalid markup. However, we may do something as in Gecko and the refactoring will make that much easier. So I'm making this depends on bug 153991. (That said, I suspect we want to spend more time on supporting valid markup than on deciding a fallback rendering for invalid markup...) 1894759 6 fred.wang 2022-08-30 02:54:56 -0700 Renaming the bug. MathML Core says to render invalid markup as an mrow, for example: "If the <mfrac> element has less or more than two in-flow children, its layout algorithm is the same as the <mrow> element. Otherwise, the first in-flow child is called numerator, the second in-flow child is called denominator and the layout algorithm is explained below." Chromium does that, and there is a plan for Firefox to align with that behavior. 1894849 7 webkit-bug-importer 2022-08-30 09:51:35 -0700 <rdar://problem/99335890> 1917556 8 fred.wang 2022-12-08 05:34:25 -0800 Firefox 106 and Chrome 109 implements the "mrow fallback" behavior described in the MathML Core specification and there is a WPT test for that: https://wpt.fyi/results/mathml/presentation-markup/mrow/mrow-fallback.html 2045797 9 harpsichen 2024-07-11 11:44:56 -0700 Pull request: https://github.com/WebKit/WebKit/pull/30705 2047625 10 ews-feeder 2024-07-22 16:17:42 -0700 Committed 281218@main (eaf4e99ac1b0): <https://commits.webkit.org/281218@main> Reviewed commits have been landed. Closing PR #30705 and removing active labels. 2047629 11 commit-queue 2024-07-22 16:25:14 -0700 Re-opened since this is blocked by bug 276917 2047639 12 aestes 2024-07-22 16:38:18 -0700 Reverted this by mistake. The revert has been undone in https://commits.webkit.org/281221@main. 215269 2013-10-27 05:25:03 -0700 2013-10-27 05:25:03 -0700 testcase error.html text/html 719 fred.wang PCFkb2N0eXBlIGh0bWw+CjxodG1sPgogIDxoZWFkPgogICAgPHRpdGxlPnRlc3RjYXNlIGVycm9y PC90aXRsZT4KICAgIDxtZXRhIGNoYXJzZXQ9InV0Zi04Ii8+CiAgICA8c2NyaXB0IHR5cGU9InRl eHQveC1tYXRoamF4LWNvbmZpZyI+CiAgICAgIE1hdGhKYXguSHViLkNvbmZpZyh7CiAgICAgICAg amF4OiBbImlucHV0L01hdGhNTCIsICJvdXRwdXQvTmF0aXZlTU1MIl0sCiAgICAgICAgZWxlbWVu dHM6IFsibWF0aGpheCJdCiAgICAgIH0pOwogICAgPC9zY3JpcHQ+CiAgICA8c2NyaXB0IHR5cGU9 InRleHQvamF2YXNjcmlwdCIgc3JjPSJodHRwOi8vY2RuLm1hdGhqYXgub3JnL21hdGhqYXgvbGF0 ZXN0L01hdGhKYXguanM/Y29uZmlnPVRlWC1BTVMtTU1MX0hUTUxvck1NTCI+PC9zY3JpcHQ+CiAg PC9oZWFkPgogIDxib2R5PgoKICAgIDxwPk5hdGl2ZSBNYXRoTUw6CiAgICAgIDxtYXRoPjxtZnJh Yz48bWk+eDwvbWk+PC9tZnJhYz48L21hdGg+IDsKICAgICAgPG1hdGg+PG1mcmFjPjxtaT54PC9t aT48bXlub2RlLz48L21mcmFjPjwvbWF0aD4KICAgIDwvcD4KCiAgICA8ZGl2IGlkPSJtYXRoamF4 Ij4KICAgICAgTWF0aEpheDoKICAgICAgPG1hdGg+PG1mcmFjPjxtaT54PC9taT48L21mcmFjPjwv bWF0aD4gOwogICAgICA8bWF0aD48bWZyYWM+PG1pPng8L21pPjxteW5vZGUvPjwvbWZyYWM+PC9t YXRoPgogICAgPC9kaXY+CgogIDwvYm9keT4KPC9odG1sPgo=