WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Add attachment
proposed patch, testcase, etc.
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.
Top of Page
Format For Printing
XML
Clone This Bug