NEW 247005
ASSERTION FAILED: !image->size().isEmpty() when setting zoom 
https://bugs.webkit.org/show_bug.cgi?id=247005
Summary ASSERTION FAILED: !image->size().isEmpty() when setting zoom
Frédéric Wang (:fredw)
Reported 2022-10-25 08:05:22 PDT
Created attachment 463215 [details] Minimized testcase See attached testcase, originally from bug comment 8. This is the same assertion as bug 206897 and bug 206579, but the repro case seems slightly different, so opening just in case. Reproduced at https://commits.webkit.org/255418@main with macos/gtk debug builds.
Attachments
Minimized testcase (315 bytes, text/html)
2022-10-25 08:05 PDT, Frédéric Wang (:fredw) 		no flags
Details
Patch (3.04 KB, patch)
2022-11-28 07:07 PST, Rob Buis 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-11-01 08:06:18 PDT
<rdar://problem/101806737>
Rob Buis
Comment 2 2022-11-28 07:07:07 PST
Created attachment 463759 [details] Patch
Nikolas Zimmermann
Comment 3 2022-12-16 07:32:51 PST
Comment on attachment 463759 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=463759&action=review > Source/WebCore/svg/graphics/SVGImageCache.cpp:60 > + if (containerSizeWithoutZoom.width() >= 1.f && containerSizeWithoutZoom.height() >= 1.f) Hmm, But a containerSizeWihtoutZoom of 0.1 is fine, no? (with a zoom factor of e.g. 10). I'm slightly confused -- I would have expected to judge if we should cache or not, based on the zoom factor? If it changes, we're going to redraw the SVGImage anyhow, right?
Note You need to log in before you can comment on or make changes to this bug.