48105 2010-10-21 18:30:11 -0700 mathml in html sometimes incorrectly parsed 2010-11-02 20:57:11 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) All All RESOLVED FIXED http://mathjax.org HTML5 P2 Normal --- 0 shogun70 webkit-unassigned abarth commit-queue davidc eric shogun70 simonjam oldest_to_newest 297775 0 shogun70 2010-10-21 18:30:11 -0700 1. visit https://eyeasme.com/Joe/MathML/HTML5_MathML_browser_test.html - there are several rendering errors 2. Look at Webkit's rendering of the Quadratic Formula - the denominator of the fraction is inline with the equation. 3. right-click on Webkit's rendering and then Inspect Element 4. the DOM for the Quadratic Formula (the output of Webkit's mathml parser) doesn't match the markup in the page (though it does seem to match what gets rendered). I've observed this in Webkit nightlies and the latest Chrome release (Vista and OS-X). This effects MathJax which can over-ride the mathml rendering, but relies on the parser being correct. 303081 1 72600 davidc 2010-11-01 16:57:37 -0700 Created attachment 72600 html5 test file with badly parsed mathml 303083 2 davidc 2010-11-01 16:58:35 -0700 I also see this in webkit nightlies (affecting native mathml rendering) and chrome 7 affecting generated DOM, thus further processing. I attach a smaller test case, nested mrows apparently being the main culprit. The first div in the attached file has <math> <mrow> <mrow><mn>1</mn></mrow> <mi>a</mi> </mrow> </math> but the innerHTML of that div, as inserted into the second, is <math> <mrow> <mrow><mn>1</mn></mrow></mrow> <mi>a</mi> </math> and the <mi>a</mi> has moved out of its correct location in the tree. 303547 3 eric 2010-11-02 12:20:03 -0700 I don't see a single <mrow> in the attached example. Can you be more clear as to what is wrong with the DOM in webkit in the attached example? 303606 4 davidc 2010-11-02 13:17:44 -0700 (In reply to comment #3) > I don't see a single <mrow> in the attached example. Can you be more clear as to what is wrong with the DOM in webkit in the attached example? ??? the full mathml expression (and the incorrect parse tree produced) is shown in comment #2. the attached example just wraps that in a complete html5 document for ease of testing. there are two mrows one inside the other but the parser apparently terminates both mrows when the inner one ends, this is not related to the mrow element, it appears that you get the same effect with any nested elements in foreign content (ie within math or svg) David 303607 5 72726 davidc 2010-11-02 13:23:00 -0700 Created attachment 72726 file showing problem descibed in comment #2 sorry! wrong file attached previously this s file intended in comment #2 and #4 303644 6 72732 simonjam 2010-11-02 14:03:44 -0700 Created attachment 72732 Patch 303672 7 72732 abarth 2010-11-02 14:35:17 -0700 Comment on attachment 72732 Patch Thanks. 303700 8 davidc 2010-11-02 14:58:46 -0700 (In reply to comment #7) > (From update of attachment 72732 [details]) > Thanks. thanks 303877 9 72732 commit-queue 2010-11-02 20:57:05 -0700 Comment on attachment 72732 Patch Clearing flags on attachment: 72732 Committed r71209: <http://trac.webkit.org/changeset/71209> 303878 10 commit-queue 2010-11-02 20:57:11 -0700 All reviewed patches have been landed. Closing bug. 72600 2010-11-01 16:57:37 -0700 2010-11-02 13:26:31 -0700 html5 test file with badly parsed mathml test3.html text/html 2444 davidc PCFkb2N0eXBlIGh0bWw+DQo8aHRtbD4NCjxoZWFkPg0KDQoNCjxzY3JpcHQgdHlwZT0idGV4dC9q YXZhc2NyaXB0Ij4NCg0KDQoNCi8vIHZhciBuc3RyMj1uc3RyZC5yZXBsYWNlKC88KFx3KykoW148 PlwvXSopXC8+L2lnLCc8JDEkMj5aWlo8LyQxPicpOw0KDQpmdW5jdGlvbiBkb21hdGgoKSB7DQog dmFyIG1tbG5vZGVhID0gZG9jdW1lbnQuZ2V0RWxlbWVudHNCeVRhZ05hbWUoIm1hdGgiKTsNCiB2 YXIgbW1sbm9kZSA9IFtdOw0KICBmb3IoIHZhciBpID0gMDsgaSA8IG1tbG5vZGVhLmxlbmd0aDsg aSsrICkgew0KICAgIG1tbG5vZGVbbW1sbm9kZS5sZW5ndGhdID0gbW1sbm9kZWFbaV07DQogIH0N CiAgZm9yICh2YXIgaT0wOyBpPG1tbG5vZGUubGVuZ3RoOyBpKyspIHsNCiAgdmFyIHBhcmVudCA9 IG1tbG5vZGVbaV0ucGFyZW50Tm9kZTsNCiAgaWYocGFyZW50IT1udWxsKXsNCiAgIHZhciBzdHI9 cGFyZW50LmlubmVySFRNTDsNCiAgIHN0ciA9IHN0ci5yZXBsYWNlKC94bWxucz0uaHR0cDpcL1wv d3d3LnczLm9yZ1wvMTk5OFwvTWF0aFwvTWF0aE1MLi9pZywnJyk7DQogICBzdHIgPSBzdHIucmVw bGFjZSgvPFw/WE1MOk5BTUVTUEFDRVtePD5dKj4vaWcsJycpOw0KICAgc3RyID0gc3RyLnJlcGxh Y2UoLzxcLyhub25lfG1wcmVzY3JpcHRzfG1nbHlwaHxtc3BhY2V8bXNsaW5lfG1hbGlnbmdyb3Vw fG1hbGlnbm1hcmspW148Pl0qPi9pZywnJyk7DQogICBzdHIgPSBzdHIucmVwbGFjZSgvPChtdGQp KFtePD5dKik+KD89XHMqPG10ZCkvaWcsJzwkMSQyPjwvJDE+Jyk7DQogICBzdHIgPSBzdHIucmVw bGFjZSgvKDxcL210ZD4pKzxcL210cj4vaWcsJzwvbXRkPjwvbXRyPicpOw0KICAgc3RyID0gc3Ry LnJlcGxhY2UoLzwobm9uZXxtcHJlc2NyaXB0c3xtZ2x5cGh8bXNwYWNlfG1zbGluZXxtYWxpZ25n cm91cHxtYWxpZ25tYXJrKShbXjw+XSopPi9pZywnPCQxJDI+PC8kMT4nKTsNCiAgIHN0ciA9IHN0 ci5yZXBsYWNlKC88XC8/KG1cdyt8bm9uZSkvaWcsZnVuY3Rpb24odyl7cmV0dXJuICh3LnRvTG93 ZXJDYXNlKCkpLnJlcGxhY2UoLyg8XC8/KS8sJyQxbTonKX0pOw0KICAgcGFyZW50LmlubmVySFRN TCA9IChzdHIpOw0KICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoIm91dCIpLmlubmVySFRNTD1z dHIucmVwbGFjZSgvPC9nLCcmbHQ7Jyk7DQogIH0NCiB9DQp9DQogIGRvY3VtZW50LndyaXRlKCI8 b2JqZWN0IGlkPVwibW1sRmFjdG9yeVwiIGNsYXNzaWQ9XCJjbHNpZDozMkY2NkEyMC03NjE0LTEx RDQtQkQxMS0wMDEwNEJEM0Y5ODdcIj48L29iamVjdD48P2ltcG9ydCBuYW1lc3BhY2U9XCJtXCIg aW1wbGVtZW50YXRpb249XCIjbW1sRmFjdG9yeVwiPz4iKTsNCiAgICB3aW5kb3cuYXR0YWNoRXZl bnQoIm9ubG9hZCIsZG9tYXRoKTsNCg0KPC9zY3JpcHQ+DQoNCjwvaGVhZD4NCjxib2R5Pg0KDQo8 ZGl2Pg0KDQoNCjxtYXRoPg0KPG1mcmFjPg0KPG1zdXA+DQo8bW4+MTwvbW4+DQo8bW4+MjwvbW4+ DQo8L21zdXA+DQo8bXNxcnQ+PG1pPmE8L21pPjwvbXNxcnQ+DQo8L21mcmFjPg0KPG1vPis8L21v Pg0KPG1pIG1hdGh2YXJpYW50PSJib2xkIj5iPC9taT4NCjxtbz4rPC9tbz4NCjxtbXVsdGlzY3Jp cHRzPg0KPG1pPkE8L21pPg0KPG5vbmUvPg0KPG1pPmk8L21pPg0KPG1wcmVzY3JpcHRzLz4NCjxu b25lLz4NCjxtaT5rPC9taT4NCjwvbW11bHRpc2NyaXB0cz4NCjxtc3BhY2Ugd2lkdGg9IjJlbSIv Pg0KPG1pPkI8L21pPg0KPC9tYXRoPg0KDQoNCjxwPmJtIDIyPC9wPg0KDQoNCjxtYXRoID4NCjxt dGFibGU+DQogIDxtdHI+DQogICAgPG10ZC8+DQogICAgPG10ZC8+DQogICAgPG10ZD48bXRleHQ+ YTwvbXRleHQ+PC9tdGQ+DQogICAgPG10ZD48bXRleHQ+YjwvbXRleHQ+PC9tdGQ+DQogICAgPG10 ZD48bXRleHQ+YzwvbXRleHQ+PC9tdGQ+DQogICAgPG10ZCBpZD0id3d3Ii8+DQogIDwvbXRyPg0K ICA8bXRyPg0KICAgIDxtdGQ+PG10ZXh0Png8L210ZXh0PjwvbXRkPg0KICAgIDxtdGQgcm93c3Bh bj0iMyI+PG1vPig8L21vPjwvbXRkPg0KICAgIDxtdGQ+PG1uPjE8L21uPjwvbXRkPjxtdGQ+PG1u PjI8L21uPjwvbXRkPjxtdGQ+PG1uPjM8L21uPjwvbXRkPg0KICAgIDxtdGQgcm93c3Bhbj0iMyI+ PG1vPik8L21vPjwvbXRkPg0KICA8L210cj4NCiAgPG10cj4NCiAgICA8bXRkPjxtdGV4dD55PC9t dGV4dD48L210ZD4NCiAgICA8bXRkPjwvbXRkPjxtdGQ+PG1uPjIyPC9tbj48L210ZD48bXRkPjxt bj4zMzwvbW4+PC9tdGQ+DQogIDwvbXRyPg0KICA8bXRyPg0KICAgIDxtdGQ+PG10ZXh0Pno8L210 ZXh0PjwvbXRkPg0KICAgIDxtdGQ+PC9tdGQ+PG10ZD48L210ZD48bXRkPjxtbj4zMzM8L21uPjwv bXRkPg0KICA8L210cj4NCjwvbXRhYmxlPg0KPC9tYXRoPg0KDQo8L2Rpdj4NCg0KPHA+cmVzdWx0 PC9wPg0KPGRpdiBpZD0ib3V0Ij4NCjwvZGl2Pg0KDQo8L2JvZHk+DQo8L2h0bWw+DQo= 72726 2010-11-02 13:23:00 -0700 2010-11-02 13:23:00 -0700 file showing problem descibed in comment #2 test3.html text/html 342 davidc PCFkb2N0eXBlIGh0bWw+DQo8aHRtbD4NCjxoZWFkPg0KPHRpdGxlPnRlc3QgMjwvdGl0bGU+DQoN CjwvaGVhZD4NCjxib2R5Pg0KDQoNCg0KPGRpdiBpZD0iYiI+DQo8L2Rpdj4NCg0KPGRpdiBpZD0i YSI+DQo8bWF0aD4NCjxtcm93Pg0KPG1yb3c+PG1uPjE8L21uPjwvbXJvdz4NCjxtaT5hPC9taT4N CjwvbXJvdz4NCjwvbWF0aD4NCjwvZGl2Pg0KDQoNCjxzY3JpcHQ+DQpkb2N1bWVudC5nZXRFbGVt ZW50QnlJZCgiYiIpLmlubmVySFRNTD1kb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiYSIpLmlubmVy SFRNTC5yZXBsYWNlKC88L2csJyZsdDsnKTsNCjwvc2NyaXB0Pg0KPC9ib2R5Pg0KPC9odG1sPg0K 72732 2010-11-02 14:03:44 -0700 2010-11-02 20:57:05 -0700 Patch bug-48105-20101102140342.patch text/plain 2786 simonjam ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCAzMTA3ZjNkMmE5NWRiNjEzMWE2NjJmMTM2YzJlZDE4ZTM3MTdiOGE3Li5lYTdiMzIy NjVhOWE2N2Y3N2VhNWViMTE4NTIyMDU3ODVhMWU5NDMxIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0 cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAK KzIwMTAtMTEtMDIgIEphbWVzIFNpbW9uc2VuICA8c2ltb25qYW1AY2hyb21pdW0ub3JnPgorCisg ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIG1hdGhtbCBpbiBo dG1sIHNvbWV0aW1lcyBpbmNvcnJlY3RseSBwYXJzZWQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndl YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTQ4MTA1CisKKyAgICAgICAgKiBodG1sNWxpYi9yZXNv dXJjZXMvd2Via2l0MDEuZGF0OiBUZXN0IGNhc2UgZm9yIGJ1Zy4KKwogMjAxMC0xMS0wMiAgSmFt ZXMgUm9iaW5zb24gIDxqYW1lc3JAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFVwZGF0ZSBwaXhl bCBiYXNlbGluZXMgZm9yIFNub3cgTGVvcGFyZCBmb3IgdGVzdHMgaW4gZmFzdC90ZXh0LyoKZGlm ZiAtLWdpdCBhL0xheW91dFRlc3RzL2h0bWw1bGliL3Jlc291cmNlcy93ZWJraXQwMS5kYXQgYi9M YXlvdXRUZXN0cy9odG1sNWxpYi9yZXNvdXJjZXMvd2Via2l0MDEuZGF0CmluZGV4IDk4Njk0NTRl N2RlNmI4N2MwZTMwMmVmNzQyYTQxNjhlODQ1NDY0MjAuLmY4ZDczZjkyNzJkODE1Y2Y0ZjQ3NWUy NDFlMTNkZmNmZTJjNmVlNWQgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL2h0bWw1bGliL3Jlc291 cmNlcy93ZWJraXQwMS5kYXQKKysrIGIvTGF5b3V0VGVzdHMvaHRtbDVsaWIvcmVzb3VyY2VzL3dl YmtpdDAxLmRhdApAQCAtNTcyLDMgKzU3MiwxOCBAQCBjb25zb2xlLmxvZygiRk9PPHNwYW4+QkFS PC9zcGFuPkJBWiIpOwogfCAgICAgPHN2ZyBzdmc+CiB8ICAgICAgIDxzdmcgdGZvb3Q+CiB8ICAg ICAgICAgPHN2ZyB0ZD4KKworI2RhdGEKKzxtYXRoPjxtcm93Pjxtcm93Pjxtbj4xPC9tbj48L21y b3c+PG1pPmE8L21pPjwvbXJvdz48L21hdGg+CisjZXJyb3JzCisjZG9jdW1lbnQKK3wgPGh0bWw+ Cit8ICAgPGhlYWQ+Cit8ICAgPGJvZHk+Cit8ICAgICA8bWF0aCBtYXRoPgorfCAgICAgICA8bWF0 aCBtcm93PgorfCAgICAgICAgIDxtYXRoIG1yb3c+Cit8ICAgICAgICAgICA8bWF0aCBtbj4KK3wg ICAgICAgICAgICAgIjEiCit8ICAgICAgICAgPG1hdGggbWk+Cit8ICAgICAgICAgICAiYSIKZGlm ZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggN2Zm ZDJlOWM2YjhhY2M2Mjk4ZTAxMmRjNzE1NmE3YmViOWYzY2U3NC4uZjAzYzllMmM1ZGIwNjZjZDRj YjYzODBiNWNmNTllMDllNGQyYTUwOSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysr IGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAxMC0xMS0wMiAgSmFtZXMg U2ltb25zZW4gIDxzaW1vbmphbUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkg Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgbWF0aG1sIGluIGh0bWwgc29tZXRpbWVzIGluY29y cmVjdGx5IHBhcnNlZAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j Z2k/aWQ9NDgxMDUKKworICAgICAgICAqIGh0bWwvcGFyc2VyL0hUTUxUcmVlQnVpbGRlci5jcHA6 CisgICAgICAgIChXZWJDb3JlOjpIVE1MVHJlZUJ1aWxkZXI6OnByb2Nlc3NFbmRUYWcpOiBGaXhl ZCB0byBtYXRjaCBIVE1MNSBzcGVjLgorCiAyMDEwLTExLTAyICBQYXZlbCBGZWxkbWFuICA8cGZl bGRtYW5AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIE5vdCByZXZpZXdlZDogV2ViIEluc3BlY3Rv cjogZm9sbG93IHVwIHRvIHI3MTEzOS4gRml4ZWQgTWFjIG1vbm9zcGFjZQpkaWZmIC0tZ2l0IGEv V2ViQ29yZS9odG1sL3BhcnNlci9IVE1MVHJlZUJ1aWxkZXIuY3BwIGIvV2ViQ29yZS9odG1sL3Bh cnNlci9IVE1MVHJlZUJ1aWxkZXIuY3BwCmluZGV4IGMzODU0Mzc1MjQ3NzkzMzQ0NjRmZDM0MjFm YmM3ZTE1ZThlOWMzMDcuLjYxMzQ2MDdjMDEwM2Q1NTJiN2VkY2IyYzA0YzlhYmU2YzFiYjliYTIg MTAwNjQ0Ci0tLSBhL1dlYkNvcmUvaHRtbC9wYXJzZXIvSFRNTFRyZWVCdWlsZGVyLmNwcAorKysg Yi9XZWJDb3JlL2h0bWwvcGFyc2VyL0hUTUxUcmVlQnVpbGRlci5jcHAKQEAgLTIyOTUsNyArMjI5 NSw4IEBAIHZvaWQgSFRNTFRyZWVCdWlsZGVyOjpwcm9jZXNzRW5kVGFnKEF0b21pY0hUTUxUb2tl biYgdG9rZW4pCiAgICAgICAgICAgICB3aGlsZSAoMSkgewogICAgICAgICAgICAgICAgIGlmIChu b2RlUmVjb3JkLT5lbGVtZW50KCktPmhhc0xvY2FsTmFtZSh0b2tlbi5uYW1lKCkpKSB7CiAgICAg ICAgICAgICAgICAgICAgIG1fdHJlZS5vcGVuRWxlbWVudHMoKS0+cG9wVW50aWxQb3BwZWQobm9k ZVJlY29yZC0+ZWxlbWVudCgpKTsKLSAgICAgICAgICAgICAgICAgICAgYnJlYWs7CisgICAgICAg ICAgICAgICAgICAgIHJlc2V0Rm9yZWlnbkluc2VydGlvbk1vZGUoKTsKKyAgICAgICAgICAgICAg ICAgICAgcmV0dXJuOwogICAgICAgICAgICAgICAgIH0KICAgICAgICAgICAgICAgICBub2RlUmVj b3JkID0gbm9kZVJlY29yZC0+bmV4dCgpOwogICAgICAgICAgICAgICAgIGlmIChub2RlUmVjb3Jk LT5lbGVtZW50KCktPm5hbWVzcGFjZVVSSSgpID09IHhodG1sTmFtZXNwYWNlVVJJKQo=