Changeset 19166 for trunk/WebCore/page/EventHandler.cpp

Timestamp:

01/26/07 16:03:56 (2 years ago)

Author:

bdakin

Message:

Reviewed by Darin.

Fix for <rdar://problem/4956565> REGRESSION: After scrolling frame,
hovering over link in this frame doesn't change cursor to pointing
hand

The mouseMove event was not being propagated correctly after using
the mouse to scroll the frame because m_mousePressed was never
getting set to false.

• page/EventHandler.cpp: (WebCore::EventHandler::handleMousePressEvent): This code does not belong here.
• page/EventHandler.h: lastEventIsMouseUp() is only ever relevant in EventHandlerMac, so it can just be a static function there.
• page/mac/EventHandlerMac.mm: (WebCore::lastEventIsMouseUp): Make this static. (WebCore::EventHandler::passMouseDownEventToWidget): Here is where we need to set m_mousePressed to false if lastEventIsMouseUp() is true.

Files:

• trunk/WebCore/page/EventHandler.cpp (modified) (1 diff)

trunk/WebCore/page/EventHandler.cpp

@@ -748 +748 @@
         else
             swallowEvent = handleMousePressEvent(mev);
-        
-        // Many AK widgets run their own event loops and consume events while the mouse is down.
-        // When they finish, currentEvent is the mouseUp that they exited on.  We need to update
-        // the khtml state with this mouseUp, which khtml never saw.
-        // If this event isn't a mouseUp, we assume that the mouseUp will be coming later.  There
-        // is a hole here if the widget consumes the mouseUp and subsequent events.
-        if (lastEventIsMouseUp())
-            m_mousePressed = false;
     }