Summary: | Crash after loading SVG fragment with XMLHttpRequest (Yosemite Map Geology Layer) | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Neumann <a.neumann> | ||||||||||
Component: | SVG | Assignee: | Alexey Proskuryakov <ap> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | mitz | ||||||||||
Priority: | P1 | ||||||||||||
Version: | 523.x (Safari 3) | ||||||||||||
Hardware: | Mac | ||||||||||||
OS: | OS X 10.4 | ||||||||||||
URL: | http://www.carto.net/neumann/webkitsvgbugs/yosemite_geology_dynamic_loading.svg | ||||||||||||
Attachments: |
|
Description
Andreas Neumann
2007-03-21 06:19:40 PDT
Created attachment 13734 [details]
crashlog for yosemite geology loading
P1 crashers need a Radar, too! :) Created attachment 13743 [details]
proposed fix
I still don't really understand why xhr.responseXML has to create a full SVG (or XHTML) DOM, but ok...
Created attachment 13744 [details]
proposed fix
Forgot to include chagnelogs.
Created attachment 13745 [details]
proposed fix
So, I didn't even notice reportWarning next to reportError :(
Thanks, Mitz!
Comment on attachment 13745 [details]
proposed fix
r=me
I think according to the style guidelines the outer ifs should have braces.
Committed revision 20363. I do not know if it's correct for d="" to emit an error message, but the crasher is fixed now. |