Show
Ignore:
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:
1 modified

Legend:

Unmodified
Added
Removed

  • trunk/WebCore/page/EventHandler.h

    r19108 r19166  
    167167    void hoverTimerFired(Timer<EventHandler>*); 
    168168 
    169     bool lastEventIsMouseUp() const; 
    170  
    171169    static bool canMouseDownStartSelect(Node*); 
    172170