Summary: | Zoom applied to embedded SVG incorrectly | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||||
Component: | SVG | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | Keywords: | InRadar | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Created attachment 47179 [details]
Patch
|
Created attachment 47176 [details] Test case To reproduce, open the attached test case in Safari, select View > Zoom Text Only, and then choose View > Zoom In a few times. Notice that the black HTML div stays put, but the blue SVG square moves and grows in size. Deselect View > Zoom Text Only. Notice that the blue SVG rectangle is still not at the top left of the black box as it should be. The issue appears to be that SVGSVGElement::currentScale() was designed for standalone SVG documents, where it’s appropriate to take the Frame’s zoom factor. For embedded SVG, this results in either applying text zoom to SVG (which is not text) or applying the full page zoom factor a second time to SVG.