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: Macintosh 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

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.