New issue
Advanced search Search tips

Issue 816320 link

Starred by 1 user
Status: Fixed
Owner:
tkent@chromium.org
Closed: Feb 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

XHTML named entities should be available in XML documents with MathML 2.0 DTD

Project Member Reported by tkent@chromium.org, Feb 26 2018 
Chrome Version: 66 canary
OS: All but iOS

What steps will reproduce the problem?
(1) Save the following XML document as foo.xml, and open it.

<!DOCTYPE math PUBLIC "-//W3C//DTD MathML 2.0//EN" "">
<math>
  &dollar;
</math>

What is the expected result?
Something like "<mathml>$</mathml>" should be shown.

What happens instead?
Parsing error is shown.
  error on line 3 at column 11: Entity 'dollar' not defined.

Please use labels and text to provide additional information.
https://html.spec.whatwg.org/multipage/xhtml.html#parsing-xhtml-documents:named-character-references

Though we don't support MathML, fixing this issue might help polyfill.

Firefox: OK
Safari: OK
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 26 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/80385fb376c4d8ddacb949f65317352977ca3f77

commit 80385fb376c4d8ddacb949f65317352977ca3f77
Author: Kent Tamura <tkent@chromium.org>
Date: Mon Feb 26 02:35:58 2018

XML Parser: XHTML named entities should be available in documents with MathML 2.0 DTD.

This CL might help polyfills though we don't support MathML 2.0.
This fixes 2,117 test failures in WPT.

Bug:  816320 
Change-Id: I93a563ae013d0c97717500130f26605e5d2ba2d3
Reviewed-on: https://chromium-review.googlesource.com/936861
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539059}
[modify] https://crrev.com/80385fb376c4d8ddacb949f65317352977ca3f77/third_party/WebKit/LayoutTests/external/wpt/html/the-xhtml-syntax/parsing-xhtml-documents/xhtml-mathml-dtd-entity-8-expected.txt
[modify] https://crrev.com/80385fb376c4d8ddacb949f65317352977ca3f77/third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp

Comment 2 by tkent@chromium.org, Feb 26 2018

Labels: M-66
Status: Fixed (was: Started)
►

Sign in to add a comment