Bug 257090

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

Ahmad Saleem
Reported 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!
Attachments
Ahmad Saleem
Comment 1 2023-05-20 11:20:35 PDT
NOTE: Changing to this: zoomFactor = 1 / renderer->view()->style().effectiveZoom(); Does not fix this testcase. :-)
Ahmad Saleem
Comment 2 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();
Radar WebKit Bug Importer
Comment 3 2023-05-27 11:03:18 PDT
Ahmad Saleem
Comment 4 2023-09-21 02:30:43 PDT
NOTE - It affects Layer Based SVG engine as well.
Note You need to log in before you can comment on or make changes to this bug.