Bug 257090
| Summary: | Include 'zoom' contributions in the getScreenCTM() result | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | fujii, sabouhallawa, simon.fraser, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | InRadar, LayerBasedSVGEngine |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
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!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
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 ***