Bug 230647

Summary: [GPU Process] `DisplayList::Recorder::getCTM` should include the device scale factor
Product: WebKit Reporter: Devin Rousso <hi>
Component: New BugsAssignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: hi, sabouhallawa, simon.fraser, thorton, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=230648
https://bugs.webkit.org/show_bug.cgi?id=230664
https://bugs.webkit.org/show_bug.cgi?id=230781
Attachments:
Description Flags
Patch none

Devin Rousso
Reported 2021-09-22 16:00:43 PDT
.
Attachments
Patch (5.04 KB, patch)
2021-10-25 16:31 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Devin Rousso
Comment 1 2021-09-22 16:06:55 PDT
(oops I think I changed the title of another bug I was gonna file and hit submit before I realized 🤣) While working on <https://webkit.org/b/230648>, I noticed that `DisplayList::Recorder::getCTM` never seems to include the device scale, meaning that if I try to use `ImageBuffer::compatibleBufferSize` and then `GraphicsContext::drawConsumingImageBuffer` it'll result in a blurry image (since the device scale used for the `ImageBuffer` is just `1`).
Radar WebKit Bug Importer
Comment 2 2021-09-29 16:01:20 PDT
<rdar://problem/83692923>
Tim Horton
Comment 3 2021-10-25 16:31:47 PDT
Created attachment 442437 [details] Patch
EWS
Comment 4 2021-10-26 14:50:35 PDT
Committed r284902 (243579@main): <https://commits.webkit.org/243579@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 442437 [details].
Note You need to log in before you can comment on or make changes to this bug.