Bug 39059

Summary: canvas drawImage does not render SVG with embedded images correctly
Product: WebKit Reporter: Justin Cormack <justin>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: ahmad.saleem792, ap, bfulgham, dino, dtrebbien, krit, mdelaney7, mike, rniwa, sabouhallawa, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac (Intel)   
OS: OS X 10.5   
See Also: https://bugs.webkit.org/show_bug.cgi?id=219770
Attachments:
Description Flags
simple svg file with embedded image
none
file that loads the svg file into canvas, renders without bitmap
none
Test case
none
test case (working) - wait for sub-resources
none
Safari 15.5 vs STP 148 none

Description Justin Cormack 2010-05-13 06:31:54 PDT
Created attachment 55968 [details]
simple svg file with embedded image

If you load an SVG image into a canvas using drawImage(), it does not draw any <image> items that are in the SVG file.

Example included - 2 files, 1 SVG file which renders correctly (3 circles and a red bitmap blob), plus Javascript that loads that image into canvas and does not show the bitmap.
Comment 1 Justin Cormack 2010-05-13 06:32:42 PDT
Created attachment 55969 [details]
file that loads the svg file into canvas, renders without bitmap
Comment 2 Dirk Schulze 2014-05-12 05:42:31 PDT
Created attachment 231289 [details]
Test case

I think this might be a duplicate of other bugs where an SVG image doesn't load embedded images.
Comment 3 Dirk Schulze 2016-10-12 05:51:27 PDT
The embedded image gets rendered on reloading the document. I am not sure if the embedded image might not be encoded in time for the onload event. I also don't know if onload requires images to be encoded and ready for rendering. But how else would the user know if the SVG is ready to render?
Comment 4 Said Abou-Hallawa 2016-10-12 10:14:44 PDT
Created attachment 291366 [details]
test case (working) - wait for sub-resources
Comment 5 Said Abou-Hallawa 2016-10-12 10:41:47 PDT
(In reply to comment #3)
> The embedded image gets rendered on reloading the document. I am not sure if
> the embedded image might not be encoded in time for the onload event. I also
> don't know if onload requires images to be encoded and ready for rendering.
> But how else would the user know if the SVG is ready to render?

I added a new test case where it waits for 100ms after onLoad() event to ensure the sub-resources are loaded/decoded before drawing the image to the canvas. Of course this will not be a reliable way to fix the issue if the data url sub-resource is big or the hardware is slow.

The data url image is the only exception to be allowed as a sub-resource in an SVG. No loading from network is involved since all the data are loaded with the SVG itself. The only overhead is decoding the the data url to an image.

The sub-resources can also be nested if the data url image is an SVG image which has a data url image.
Comment 6 Ahmad Saleem 2022-06-29 15:51:09 PDT
If I use the attached test case [test case(working) - wait for sub-resources], it renders "red" fog pixelated in Safari 15.5 on macOS 12.4 but when using Safari Technical Preview 148, it renders it similar to other browsers (Chrome Canary 105 and Firefox Nightly 104).

I think it is fixed along the lines. Can this be marked as "RESOLVED CONFIGURATION CHANGED" considering it is fixed in future versions? If someone knows exact bug, where it is fixed or likely fixed as knock-off effect then it can be marked as Duplicate of XYZ. Thanks!
Comment 7 Alexey Proskuryakov 2022-06-29 16:46:19 PDT
I think that the bug will be fixed once Dirk's "Test case" passes, and that still doesn't, at least not in a clean browsing session.
Comment 8 Ahmad Saleem 2022-06-29 17:41:17 PDT
Created attachment 460563 [details]
Safari 15.5 vs STP 148

@ap - I tested it in "Private Browse" where cookies and cache does not impact and this attached is screenshot of STP 148 behavior compared to Safari 15.5. STP behavior matches with Chrome Canary 105 and Firefox Nightly 104. If I am testing it incorrectly, please educate, so in future I don't make those mistakes. Thanks!
Comment 9 Alexey Proskuryakov 2022-07-01 10:51:04 PDT
I think that you tested with "test case (working) - wait for sub-resources", but not with "Test case"? As Said explained when posting it, that was a demonstration meant to help isolate what the bug is - it always passed, in contrast to the actual test case for the bug.

When loading the actual test case in a clean session, there are only the three overlapping circles displayed.
Comment 10 Brent Fulgham 2022-07-15 14:49:59 PDT
Safari, Chrome, and Firefox all agree on rendering for this test case. I don't believe there is any remaining compatibility issue.
Comment 11 Brent Fulgham 2022-07-15 14:50:51 PDT
Actually -- I see the issue: There is a race condition. If you reload the SVG test case it works. The first load lacks the background image.
Comment 12 Radar WebKit Bug Importer 2022-07-15 14:51:04 PDT
<rdar://problem/97095682>