RESOLVED FIXED 231385
BifurcatedGraphicsContext doesn't bifurcate draw{Tiled}Image 
https://bugs.webkit.org/show_bug.cgi?id=231385
Summary BifurcatedGraphicsContext doesn't bifurcate draw{Tiled}Image
Tim Horton
Reported 2021-10-07 12:49:15 PDT
BifurcatedGraphicsContext doesn't bifurcate drawTiledImage
Attachments
Patch (11.76 KB, patch)
2021-10-07 12:52 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (14.61 KB, patch)
2021-10-07 13:14 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Tim Horton
Comment 1 2021-10-07 12:52:00 PDT
Created attachment 440529 [details] Patch
Tim Horton
Comment 2 2021-10-07 12:52:03 PDT
<rdar://problem/83922633>
Tim Horton
Comment 3 2021-10-07 12:54:44 PDT
Actually, wait... drawImage() has the same problem.
Tim Horton
Comment 4 2021-10-07 13:14:19 PDT
Created attachment 440531 [details] Patch
Devin Rousso
Comment 5 2021-10-07 13:43:37 PDT
Comment on attachment 440531 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=440531&action=review r=me > Source/WebCore/platform/graphics/BifurcatedGraphicsContext.h:102 > + ImageDrawResult drawImage(Image&, const FloatRect& destination, const FloatRect& source, const ImagePaintingOptions& = { ImageOrientation::FromImage }) final; Aside: it's kinda odd that the other `drawImage` are also `virtual` even though they just call into one of the other `drawImage`
Devin Rousso
Comment 6 2021-10-07 13:46:43 PDT
Comment on attachment 440531 [details] Patch btw do we also need to do `drawImageBuffer` (and maybe somehow `drawConsumingImageBuffer`)?
Tim Horton
Comment 7 2021-10-07 13:51:42 PDT
(In reply to Devin Rousso from comment #5) > Comment on attachment 440531 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=440531&action=review > > r=me > > > Source/WebCore/platform/graphics/BifurcatedGraphicsContext.h:102 > > + ImageDrawResult drawImage(Image&, const FloatRect& destination, const FloatRect& source, const ImagePaintingOptions& = { ImageOrientation::FromImage }) final; > > Aside: it's kinda odd that the other `drawImage` are also `virtual` even > though they just call into one of the other `drawImage` I agree! Need to check if that is just an oversight (of mine, from June) or if something (DisplayList?) actually cares.(In reply to Devin Rousso from comment #6) > Comment on attachment 440531 [details] > Patch > > btw do we also need to do `drawImageBuffer` (and maybe somehow > `drawConsumingImageBuffer`)? These *do* eventually just turn into drawNativeImage, so they should be OK. Can't get a GeneratedImage-backed ImageBuffer. Agree that it looks funny though.
EWS
Comment 8 2021-10-07 15:18:57 PDT
Committed r283747 (242669@main): <https://commits.webkit.org/242669@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 440531 [details].
Note You need to log in before you can comment on or make changes to this bug.