Bug 16517

Summary: SVGs with a viewbox specified do not seem to work as SVGImages
Product: WebKit Reporter: Beth Dakin <bdakin>
Component: SVGAssignee: Nikolas Zimmermann <zimmermann>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, krit
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
HTML of test
none
SVG for test
none
Patch rwlbuis: review+

Beth Dakin
Reported 2007-12-19 14:30:24 PST
SVGs with a viewbox specified do not work when used in an <img> tag or when used as an SVG image. See attached reduction. <rdar://problem/5656379>
Attachments
HTML of test (228 bytes, text/html)
2007-12-19 14:31 PST, Beth Dakin 		no flags
Details
SVG for test (177 bytes, image/svg+xml)
2007-12-19 14:32 PST, Beth Dakin 		no flags
Details
Patch (18.20 KB, patch)
2011-06-01 08:19 PDT, Nikolas Zimmermann 		rwlbuis: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Beth Dakin
Comment 1 2007-12-19 14:31:47 PST
Created attachment 17995 [details] HTML of test
Beth Dakin
Comment 2 2007-12-19 14:32:12 PST
Created attachment 17996 [details] SVG for test
Dirk Schulze
Comment 3 2011-05-18 10:41:30 PDT
This example can't work, since the referenced SVG is not accessible by calling black.svg. Can you add a new HTML file with the correct link please?
Alexey Proskuryakov
Comment 4 2011-05-19 00:51:14 PDT
I think that the attached test should work if you download the files and open it locally.
Nikolas Zimmermann
Comment 5 2011-06-01 08:19:54 PDT
Created attachment 95603 [details] Patch This works nowadays, adding the testcase.
Rob Buis
Comment 6 2011-06-01 08:28:51 PDT
Comment on attachment 95603 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=95603&action=review LGTM > LayoutTests/ChangeLog:8 > + SVGs with a viewBox work as background image nowadays, add a testcase covering it, orignally from Beth Dakin. s/orignally/originally
Nikolas Zimmermann
Comment 7 2011-06-01 09:08:50 PDT
Landed in r87807.
Note You need to log in before you can comment on or make changes to this bug.