Bug 5278

Summary: WARNING: Malformed viewbox string: (l: 0)
Product: WebKit Reporter: Julien Palmas <julien.palmas>
Component: SVGAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P4    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.carto.net/papers/svg/samples/symbol.svg
Attachments:
Description Flags
Reduced test case.
none
Another reduced test case
none
rwlbuis patch eric: review+

Julien Palmas
Reported 2005-10-05 16:01:23 PDT
The URL points to a svg that triggers this warning. This bug might already exist in the bugzilla list, but could not find where...
Attachments
Reduced test case. (443 bytes, image/svg+xml)
2005-10-05 18:04 PDT, Eric Seidel (no email) 		no flags
Details
Another reduced test case (293 bytes, text/plain)
2005-10-06 07:02 PDT, Julien Palmas 		no flags
Details
rwlbuis patch (1.66 KB, patch)
2005-10-10 13:28 PDT, Julien Palmas 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2005-10-05 18:04:30 PDT
Created attachment 4227 [details] Reduced test case. the use of <symbol> seems to be related. This test might not be fully reduced yet though.
Julien Palmas
Comment 2 2005-10-06 07:02:49 PDT
Created attachment 4234 [details] Another reduced test case This test case also triggers the warning. If you add a viewBox attribute to the symbol element, the warning disappears . For example : <symbol id="coniferous" viewBox="0 0 100 50"> The warning comes from KSVG::SVGFitToViewBoxImpl::parseViewBox(KDOM::DOMStringImpl *str) The viewBox attribute is not compulsory (I think), but the parseViewBox method is always called. This should be modified.
Julien Palmas
Comment 3 2005-10-06 07:06:19 PDT
Ah, forgot something. If you add the viewBox attribute like I said before, the warning disappears, but the <use> element is not rendered anymore ...
Julien Palmas
Comment 4 2005-10-10 13:28:26 PDT
Created attachment 4299 [details] rwlbuis patch
Eric Seidel (no email)
Comment 5 2005-10-10 14:03:44 PDT
Comment on attachment 4299 [details] rwlbuis patch I'm not sure I really like this "dummy" (hack) SVG element for use, but at least this patch fixes the hack.
Rob Buis
Comment 6 2005-10-11 04:01:38 PDT
It seems there is still a problem with the reduced testcase. My suspicion, aspect ratio is not correctly handled! I'll look at it tonight, unless someone beats me to it :)
Note You need to log in before you can comment on or make changes to this bug.