RESOLVED WONTFIX 117443
Consider using HTML parser to parse SVG
https://bugs.webkit.org/show_bug.cgi?id=117443
Summary Consider using HTML parser to parse SVG
Ryosuke Niwa
Reported 2013-06-10 16:43:39 PDT
https://chromium.googlesource.com/chromium/blink/+/b69618018614278cda72077611adc093f460dc57 Parse SVG as HTML This change switches the SVG parser to be the HTML parser all the time. This work is being done behind a flag to gauge the potential compat risk. This is an interesting experiment.
Attachments
Dirk Schulze
Comment 1 2014-07-12 23:30:19 PDT
The experiment kind of failed. HTML parser does not allow entities. There are thousands of SVGs out there which use custom XML entities. (Most exported by Illustrator :(.) Either the HTML parser allows a legacy mode with entities or we can not proceed with this idea.
Philip Rogers
Comment 2 2014-07-13 11:43:35 PDT
(In reply to comment #1) > The experiment kind of failed. > > HTML parser does not allow entities. There are thousands of SVGs out there which use custom XML entities. (Most exported by Illustrator :(.) > > Either the HTML parser allows a legacy mode with entities or we can not proceed with this idea. I discussed this with Kouhei and we don't think a basic level of xml entity support would be too hard to implement in the html parser.
Dirk Schulze
Comment 3 2014-07-13 22:24:19 PDT
(In reply to comment #2) > (In reply to comment #1) > > The experiment kind of failed. > > > > HTML parser does not allow entities. There are thousands of SVGs out there which use custom XML entities. (Most exported by Illustrator :(.) > > > > Either the HTML parser allows a legacy mode with entities or we can not proceed with this idea. > > I discussed this with Kouhei and we don't think a basic level of xml entity support would be too hard to implement in the html parser. Does the HTML parser handle renaming of ns shorthands? Instead of xmlns:xlink="http://www.w3.org/1999/xlink" this xmlns:l="http://www.w3.org/1999/xlink" and l:href This was another pattern we saw.
Ryosuke Niwa
Comment 4 2014-07-14 08:44:02 PDT
As far as I know, this experiment failed due to XSLT use.
Note You need to log in before you can comment on or make changes to this bug.