290522 - page scrolls in wrong direction (seamonkey/FF)

Status: RESOLVED FIXED

(Core :: Web Painting, defect)

Description

[Peter van der Woude [:Peter6]]

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b2) Gecko/20050415
Firefox/1.0+

repro:
1.Open FF
2.Open testcase
3.scroll the page down by pressing a few times on the scrolldown button on the
vertical scrollbar
4.place the mouse just above the slider and click
5.the page scrolls down i/o up
6.press a few times on the scrollright button on the horizontal scrollbar
7.place the mouse just left of the slider and click
8.the page is scrolled right i/o left

testcase coming up

Comment 1

[Peter van der Woude [:Peter6]]

Attachment: testcase

Comment 2

[WADA:World Anti-bad-Duping Agency]

(In reply to comment #0)
> Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b2) Gecko/20050415
> Firefox/1.0+

> 4.place the mouse just above the slider and click
> 5.the page scrolls down i/o up

> 7.place the mouse just left of the slider and click
> 8.the page is scrolled right i/o left

Mouse position is invalidly reported in some cases on 2005-04-15 builds?
See Bug 290494 for Thunderbird case.

Comment 3

[Steve England [:stevee]]

*** Bug 290607 has been marked as a duplicate of this bug. ***

Comment 4

[Steve England [:stevee]]

*** Bug 290597 has been marked as a duplicate of this bug. ***

Comment 5

[Peter van der Woude [:Peter6]]

this regressed between 20050414 and 20050415.

related to Bug 290553 ?

Comment 6

[Jerry Baker]

Probably. All of these bugs certainly make the checkin for bug 289792 look very
suspicious. Anyone have a local tree they can back out that checkin from and see
what happens?

Comment 7

[Benoît]

How can my bug (bug#290597) be a duplicate of this bug when my bug is older?!

>_<

This offends me as

-it's wrong
-I did search first
-I don't like reporting a duplicate

Comment 8

[Benoît]

Oops, sorry for my stupidity. Apparently I got confused with the bug numbers,
and also took the "last updated" hours as reference.

My apologies.

Comment 9

[Boris Zbarsky [:bzbarsky]]

Attachment: Fix

This fixes all the problems I could find with scrollbars... it's really rather
a miracle this code ever worked before; getting the event->point from an
nsEvent that comes from a DOMEvent inside DOM event dispatch is not
particularly guaranteed to give you anything useful in _your_ coordinates....

Comment 10

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 180983 [details] [diff] [review]
Fix

Requesting approval for 1.8b2 for this regression fix.

Comment 11

[Brendan Eich [:brendan]]

Comment on attachment 180983 [details] [diff] [review]
Fix

a=brendan for 1.8b2.

/be

Comment 12

[Boris Zbarsky [:bzbarsky]]

Fixed.

Comment 13

[Steve England [:stevee]]

*** Bug 290832 has been marked as a duplicate of this bug. ***

Comment 14

[José Jeria]

Did this fix cause bug 290921?