Bug 257090 - Include 'zoom' contributions in the getScreenCTM() result
Summary: Include 'zoom' contributions in the getScreenCTM() result
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar, LayerBasedSVGEngine
Depends on:
Blocks:
 
Reported: 2023-05-20 11:02 PDT by Ahmad Saleem
Modified: 2023-10-03 14:48 PDT (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.