484677 - Mouse coordinates in zoomed foreignObject are calculated incorrectly

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Kai.Londenberg]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2a1pre) Gecko/20090223 Minefield/3.2a1pre
Build Identifier: validated for 3.0.6 to Minefield/3.2a1pre

When a page is zoomed, coordinates of mouse events don't match the content in foreignObject elements.

I guess the zoom transformation somehow gets applied twice.

If you enhance the page zoom level, the "sensitive region" of the link gets too large. If you reduce the zoom level, the "sensitive region" of the link gets too small.

To reproduce the problem, take the simple XUL / SVG Page below, and load it into Firefox. Then zoom the page to a large zoom level (for example using ctrl+mousewheel) and check the marked boundaries of the link against the area in which the cursor changes since it hovers above it (sensitive region). 



--
 <xul:page  width="900" height="900" xmlns:svg="http://www.w3.org/2000/svg" 
xmlns:html="http://www.w3.org/1999/xhtml" xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"> 
            <svg:svg width="800px" height="800px" >
                <svg:foreignObject width="420" height="80" >
                     <html:a style="display:inline; border: 1px solid grey;" href="http://www.google.de/">Google</html:a>
                </svg:foreignObject>
            </svg:svg>
</xul:page>
--


Reproducible: Always

Steps to Reproduce:
1.Load the XUL/SVG Page from bug description into Firefox
2.Zoom the page to a large zoom level using ctrl+mousewheel
3.Move the mouse over the link. The sensitive region of the link is too large.
Actual Results:  
The sensitive region of the link is too large. Mouse coordinates don't match up with the contents of the foreignObject

Expected Results:  
The sensitive region should scale along correctly

Comment 1

[Robert Longson [:longsonr]]

Attachment: reporters testcase as attachment

Comment 2

[Robert Longson [:longsonr]]

Attachment: patch

Comment 3

[Robert Longson [:longsonr]]

Attachment: update patch [Checkin: See comment 7 & 10+12]

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 368922 [details] [diff] [review]
update patch
[Checkin: See comment 7 & 10+12]

Should be able to use reftest-zoom and elementFromPoint to write an easy test here

Comment 5

[Robert Longson [:longsonr]]

Attachment: reftest [Checkin: Comment 7]

Comment 6

[Robert Longson [:longsonr]]

Attachment: reftest for 1.9.1 [Checkin: Comment 11]

Comment 7

[Robert Longson [:longsonr]]

checked in http://hg.mozilla.org/mozilla-central/rev/b48ae9fb7c0c and http://hg.mozilla.org/mozilla-central/rev/e2531d6ba033

Comment 8

[Robert Longson [:longsonr]]

Comment on attachment 368922 [details] [diff] [review]
update patch
[Checkin: See comment 7 & 10+12]

Low risk patch, includes a reftest

Comment 9

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 368922 [details] [diff] [review]
update patch
[Checkin: See comment 7 & 10+12]

a191=beltzner for this and the associated reftest

Comment 10

[Serge Gautherie (:sgautherie)]

Comment on attachment 368922 [details] [diff] [review]
update patch
[Checkin: See comment 7 & 10+12]


http://hg.mozilla.org/releases/mozilla-1.9.1/rev/6676b7742b23

After fixing context for
{
patching file layout/svg/base/src/nsSVGForeignObjectFrame.cpp
Hunk #7 FAILED at 677
1 out of 7 hunks FAILED
}

Comment 11

[Serge Gautherie (:sgautherie)]

Comment on attachment 369192 [details] [diff] [review]
reftest for 1.9.1
[Checkin: Comment 11]


http://hg.mozilla.org/releases/mozilla-1.9.1/rev/c2d3a50e3dbf

Comment 12

[Serge Gautherie (:sgautherie)]

Comment on attachment 368922 [details] [diff] [review]
update patch
[Checkin: See comment 7 & 10+12]


http://hg.mozilla.org/releases/mozilla-1.9.1/rev/958454f9cd0e
(bad merge fix)