Bug 244915

Summary: TextureMapperGL::beginRoundedRectClip should use TransformationMatrix::mapQuad instead of TransformationMatrix::projectQuad
Product: WebKit Reporter: Fujii Hironori <Hironori.Fujii>
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

Description Fujii Hironori 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.
Comment 1 Fujii Hironori 2022-09-07 17:34:18 PDT 
Created attachment 462191 [details]
WIP patch
Comment 2 Fujii Hironori 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.
Comment 3 Fujii Hironori 2022-09-07 17:37:21 PDT 
Created attachment 462193 [details]
test case
Comment 4 Radar WebKit Bug Importer 2022-09-14 17:34:19 PDT 
<rdar://problem/99945034>