Bug 257090 – Include 'zoom' contributions in the getScreenCTM() result

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: SVG

[Ahmad Saleem]

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!

[Ahmad Saleem]

NOTE: Changing to this:

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

Does not fix this testcase. :-)

[Ahmad Saleem]

(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]

<rdar://problem/109931108>

[Ahmad Saleem]

NOTE - It affects Layer Based SVG engine as well.

[Ahmad Saleem]

This test would be good addition to WPT (we should ask Chromium folks to push it) or create new one ourselves.

[Ahmad Saleem]

I fixed this with bug 308970 and added test as well.

*** This bug has been marked as a duplicate of bug 308970 ***