480337 - zoom is broken

Status: RESOLVED WORKSFORME

(Firefox :: General, defect)

Description

[Peter van der Woude [:Peter6]]

repro 1:
open url
put the pointer in the input to the right of the text "Ga naar" (bottom-right)
type "Amsterdam" and hit enter
scroll the mousewheel

repro 2:
drag the slider on the left side up and down.

result:
you can zoom in one step, if you're lucky, but it zooms out most of the time.

regressionwindow:
works in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090113 Minefield/3.2a1pre ID:20090113035246
broken in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090114 Minefield/3.2a1pre ID:20090114034711

Comment 1

[Peter van der Woude [:Peter6]]

Attachment: screenshot

also, if you have the Amsterdam map and click anywhere a field should be highlighted and a balloon with a W should show up.

Comment 2

[Steve England [:stevee]]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090226 Shiretoko/3.1b3pre

Zooming with both the mouse-wheel and the [+] [-] icons works fine for me on the branch FWIW.

Comment 3

[Peter van der Woude [:Peter6]]

checkins
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9dbded90af2a&tochange=89811ac1b35a

Comment 4

[Peter van der Woude [:Peter6]]

Boris,

might this be caused by Bug 449780 (I'm gambling)

Comment 5

[Boris Zbarsky [:bzbarsky]]

Pretty unlikely.  The style reresolution fixes, getElementsByClassName change, and various other stuff in that range seems more likely.

I should note that using the steps in comment 0 zooming in works fine for me on mac...  Otherwise I'd offer to bisect to find the changeset that caused the problem.

Comment 6

[Peter van der Woude [:Peter6]]

hmmm, this was actually RESOLVED between 20090227 and 20090228

http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=cc8d4e656113&tochange=f3d5f4a980a0

I don't know which patch did it, so setting WFM untill someone else has a better idea.