Bug 257090

Summary:	Include 'zoom' contributions in the getScreenCTM() result
Product:	WebKit	Reporter:	Ahmad Saleem <ahmad.saleem792>
Component:	SVG	Assignee:	Nobody <webkit-unassigned>
Status:	NEW ---
Severity:	Normal	CC:	Hironori.Fujii, sabouhallawa, simon.fraser, webkit-bug-importer, zimmermann
Priority:	P2	Keywords:	InRadar, LayerBasedSVGEngine
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Ahmad Saleem 2023-05-20 11:02:41 PDT

Hi Team,

While going through Blink's commit, I came across another failing testcase:

Testcase - https://jsfiddle.net/Lwrx9jf1/show

^ STP170 shows 'assert_approx_equals: a expected 2 +/- 0.000005 but got 1
'

Blink Commit - https://chromium.googlesource.com/chromium/src/+/7254890a169ba91d8ad6f61374cd93ec59029441

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/svg/SVGSVGElement.cpp#436

_______

Just wanted to raise, so we can fix it.

Thanks!

Comment 1 Ahmad Saleem 2023-05-20 11:20:35 PDT

NOTE: Changing to this:

zoomFactor = 1 / renderer->view()->style().effectiveZoom();

Does not fix this testcase. :-)

Comment 2 Ahmad Saleem 2023-05-20 11:21:03 PDT

(In reply to Ahmad Saleem from comment #1)
> NOTE: Changing to this:
> 
> zoomFactor = 1 / renderer->view()->style().effectiveZoom();
> 
> Does not fix this testcase. :-)

Sorry this:

zoomFactor = 1 / renderer->view().style().effectiveZoom();

Comment 3 Radar WebKit Bug Importer 2023-05-27 11:03:18 PDT

<rdar://problem/109931108>

Comment 4 Ahmad Saleem 2023-09-21 02:30:43 PDT

NOTE - It affects Layer Based SVG engine as well.