Bug 16517 - SVGs with a viewbox specified do not seem to work as SVGImages
Summary: SVGs with a viewbox specified do not seem to work as SVGImages
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nikolas Zimmermann
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2007-12-19 14:30 PST by Beth Dakin
Modified: 2011-06-01 09:08 PDT (History)
2 users (show)

See Also:

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+
Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Beth Dakin 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>
Comment 1 Beth Dakin 2007-12-19 14:31:47 PST 
Created attachment 17995 [details]
HTML of test
Comment 2 Beth Dakin 2007-12-19 14:32:12 PST 
Created attachment 17996 [details]
SVG for test
Comment 3 Dirk Schulze 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?
Comment 4 Alexey Proskuryakov 2011-05-19 00:51:14 PDT 
I think that the attached test should work if you download the files and open it locally.
Comment 5 Nikolas Zimmermann 2011-06-01 08:19:54 PDT 
Created attachment 95603 [details]
Patch

This works nowadays, adding the testcase.
Comment 6 Rob Buis 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
Comment 7 Nikolas Zimmermann 2011-06-01 09:08:50 PDT 
Landed in r87807.