112448 - Mousewhell doesn't work if window not on main monitor

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Mike Pinkerton (not reading bugmail)]

- open a window on the 2nd monitor with a url that scrolls
- use mousewheel

expected:
- page scrolls

actual:
- nothing happens

something regressed, this worked a week or two ago. what changed?! this works
fine if the window is on the main monitor.

Comment 1

[Mike Pinkerton (not reading bugmail)]

taking. i have a patch

Comment 2

[Mike Pinkerton (not reading bugmail)]

patch

Index: nsMacWindow.cpp
===================================================================
RCS file: /cvsroot/mozilla/widget/src/mac/nsMacWindow.cpp,v
retrieving revision 1.105
diff -u -2 -r1.105 nsMacWindow.cpp
--- nsMacWindow.cpp     2001/10/27 07:38:03     1.105
+++ nsMacWindow.cpp     2001/11/28 20:47:21
@@ -679,6 +679,10 @@
   if ( err1 == noErr && err2 == noErr && err3 == noErr ) {
     nsMacWindow* self = NS_REINTERPRET_CAST(nsMacWindow*, userData);
-    if ( self )
+    if ( self ) {
+      // convert mouse to local coordinates since that's how gecko wants them
+      StPortSetter portSetter(self->mWindowPtr);
+      ::GlobalToLocal(&mouseLoc);
       self->mMacEventHandler->Scroll ( axis, delta, mouseLoc );
+    }
   }
   return noErr;

Comment 3

[Mike Pinkerton (not reading bugmail)]

needing r/sr. cc'ing bryner.

Comment 4

[Simon Fraser [no longer active]]

sr=sfraser

Comment 5

[Mike Pinkerton (not reading bugmail)]

fixed.