547596 - SVG zoom events don't fire ?

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Duncan Loveday]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.2) Gecko/20100115 Firefox/3.6 (.NET CLR 2.0.50727)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.2) Gecko/20100115 Firefox/3.6 (.NET CLR 2.0.50727)

In the attached, the function zoomHandler() named in the SVG onzoom attribute is not called when currentScale is changed (worked on 3.5.x)

Reproducible: Always

Steps to Reproduce:
1. Open the html attachment and click the button
2.
3.
Actual Results:  
Two alerts are shown, one when the SVG is loaded and another when the onclick handler for the button is called.

Expected Results:  
A total of three alerts should be shown, the two described above and another when the onzoom handler for the SVG is called.

Comment 1

[Duncan Loveday]

Attachment: Test SVG

Comment 2

[Duncan Loveday]

Attachment: Test html

Comment 3

[Duncan Loveday]

Attachment: Test SVG v2 (with content)

Comment 4

[Duncan Loveday]

Attachment: Test html v2 (improved alert messages)

Comment 5

[Robert Longson [:longsonr]]

If it's a regression do you fancy getting a regression range?

Comment 6

[Duncan Loveday]

Yes I can do that.

Renesis plugin has the same behaviour BTW - no chance it's an intentional change is there ?

Comment 7

[Robert Longson [:longsonr]]

Never mind. I don't need a regression range any more. The issue is obvious enough.
It's not an intentional change.

Comment 8

[Robert Longson [:longsonr]]

Attachment: patch

Comment 9

[Jonathan Watt [:jwatt]]

Comment on attachment 428095 [details] [diff] [review]
patch

r=jwatt

Comment 10

[Duncan Loveday]

This is quick work !

BTW is it safe to assume this fix WON'T be released until 3.7.x ?

I can code round it but calling the zoom handler explicitly but that'll break if the behaviour changes in a minor update

Comment 11

[Jonathan Watt [:jwatt]]

3.7 hasn't been released. Do you mean 3.6.x? This patch will probably be in the former, but won't be in the latter.

Comment 12

[Duncan Loveday]

Yes, that's what I meant - patch won't be in a minor update for 3.6.x. I'll code up a bit of nasty user agent sniffing for 3.6 (but not 3.7).

Thanks Jonathan

Comment 13

[Robert Longson [:longsonr]]

pushed http://hg.mozilla.org/mozilla-central/rev/13489c44f99d