Bug 244915

Summary: TextureMapperGL::beginRoundedRectClip should use TransformationMatrix::mapQuad instead of TransformationMatrix::projectQuad
Product: WebKit Reporter: Fujii Hironori <fujii.hironori>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: cmarcelo, ews-watchlist, kondapallykalyan, luiz, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
WIP patch
ews-feeder: commit-queue-
wrong test case (wrong explanation in the test)
none
test case none

Fujii Hironori
Reported 2022-09-07 17:33:32 PDT
TextureMapperGL::beginRoundedRectClip should use TransformationMatrix::mapQuad instead of TransformationMatrix::projectQuad TextureMapperGL::beginRoundedRectClip and TextureMapperGL::beginScissorClip are using projectQuad. But, projectQuad is used mainly for hit testing by using a inversed matrix. projectQuad produces the same results with mapQuad if the transformation matrix has not z-axis transformation. I think there is no actual problem for beginRoundedRectClip and beginScissorClip to use projectQuad. But, it should be mapQuad.
Attachments
WIP patch (1.22 KB, patch)
2022-09-07 17:34 PDT, Fujii Hironori 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
wrong test case (wrong explanation in the test) (912 bytes, text/html)
2022-09-07 17:36 PDT, Fujii Hironori 		no flags
Details
test case (912 bytes, text/html)
2022-09-07 17:37 PDT, Fujii Hironori 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Fujii Hironori
Comment 1 2022-09-07 17:34:18 PDT
Created attachment 462191 [details] WIP patch
Fujii Hironori
Comment 2 2022-09-07 17:36:22 PDT
Created attachment 462192 [details] wrong test case (wrong explanation in the test) Because mapQuad has a problem at the moment (bug#244824), this patch introduces a regression in case of w<0.
Fujii Hironori
Comment 3 2022-09-07 17:37:21 PDT
Created attachment 462193 [details] test case
Radar WebKit Bug Importer
Comment 4 2022-09-14 17:34:19 PDT
<rdar://problem/99945034>
Note You need to log in before you can comment on or make changes to this bug.