WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 16164
SVGs with width or height of 100% hang when used as a background-image
https://bugs.webkit.org/show_bug.cgi?id=16164
Summary
SVGs with width or height of 100% hang when used as a background-image
Beth Dakin
Reported
2007-11-27 16:18:37 PST
<
rdar://problem/5616052
>
Attachments
Test case
(231 bytes, text/html)
2007-11-27 16:19 PST
,
Beth Dakin
no flags
Details
Sample SVG
(124 bytes, image/svg+xml)
2007-11-27 16:19 PST
,
Beth Dakin
no flags
Details
Patch
(5.16 KB, patch)
2007-11-27 22:23 PST
,
Beth Dakin
oliver
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Beth Dakin
Comment 1
2007-11-27 16:19:24 PST
Created
attachment 17562
[details]
Test case
Beth Dakin
Comment 2
2007-11-27 16:19:51 PST
Created
attachment 17563
[details]
Sample SVG
Beth Dakin
Comment 3
2007-11-27 22:23:23 PST
Created
attachment 17566
[details]
Patch This fixes the hang appropriately. There is still a bug that the actual SVG does not render, but that is an existing problem. (I say it is "existing" because using this SVG in an <img> does NOT cause a hang, but it does cause the rendering issue, so this patch strictly addresses the hang.) The rendering issue is filed in radar, and I will file it shortly in Bugzilla as well.
Oliver Hunt
Comment 4
2007-11-27 22:29:34 PST
Comment on
attachment 17566
[details]
Patch Woo! beth is a l33t svg hacker now :D
Beth Dakin
Comment 5
2007-11-27 22:29:51 PST
Rendering issue filed as
http://bugs.webkit.org/show_bug.cgi?id=16167
Beth Dakin
Comment 6
2007-11-27 22:32:04 PST
Yay! Fixed with
r28102
.
Eric Seidel (no email)
Comment 7
2007-11-28 12:18:45 PST
FYI: This was also in bugzilla as
bug 15638
and I think
bug 16167
was the reason why the patch on
bug 15638
never landed.
Eric Seidel (no email)
Comment 8
2007-11-28 12:20:56 PST
***
Bug 15638
has been marked as a duplicate of this bug. ***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug