Bug 33988

Summary: Zoom applied to embedded SVG incorrectly
Product: WebKit Reporter: mitz
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal Keywords: InRadar
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Test case
none
Patch darin: review+

Description mitz 2010-01-21 22:00:29 PST
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.
Comment 1 mitz 2010-01-21 22:01:27 PST
<rdar://problem/7568696>
Comment 2 mitz 2010-01-21 23:54:39 PST
Created attachment 47179 [details]
Patch
Comment 3 mitz 2010-01-22 00:20:57 PST
Fixed in <http://trac.webkit.org/projects/webkit/changeset/53684>.