[webkit-changes] cvs commit: WebCore/khtml khtmlview.cpp

[Darin]

darin       05/12/05 17:10:37

  Modified:    .        ChangeLog
               khtml    khtmlview.cpp
  Log:
          Reviewed by Eric.
  
          - fixed <rdar://problem/4364374> Denver Regression: ModBlog.com user menus don't stay visible due to extra mouseout events (5668)
            http://bugzilla.opendarwin.org/show_bug.cgi?id=5668
  
          * khtml/khtmlview.cpp: (KHTMLView::dispatchMouseEvent): Do the same "skip text node"
          logic for the "oldUnder" variable as for the "targetNode" variable.
  
  Revision  Changes    Path
  1.477     +10 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.476
  retrieving revision 1.477
  diff -u -r1.476 -r1.477
  --- ChangeLog	6 Dec 2005 00:23:39 -0000	1.476
  +++ ChangeLog	6 Dec 2005 01:10:31 -0000	1.477
  @@ -1,3 +1,13 @@
  +2005-12-05  Darin Adler  <darin at apple.com>
  +
  +        Reviewed by Eric.
  +
  +        - fixed <rdar://problem/4364374> Denver Regression: ModBlog.com user menus don't stay visible due to extra mouseout events (5668)
  +          http://bugzilla.opendarwin.org/show_bug.cgi?id=5668
  +
  +        * khtml/khtmlview.cpp: (KHTMLView::dispatchMouseEvent): Do the same "skip text node"
  +        logic for the "oldUnder" variable as for the "targetNode" variable.
  +
   2005-12-05  John Sullivan  <sullivan at apple.com>
   
           Reviewed by Darin Adler.
  
  
  
  1.157     +2 -0      WebCore/khtml/khtmlview.cpp
  
  Index: khtmlview.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/khtmlview.cpp,v
  retrieving revision 1.156
  retrieving revision 1.157
  diff -u -r1.156 -r1.157
  --- khtmlview.cpp	4 Dec 2005 11:35:03 -0000	1.156
  +++ khtmlview.cpp	6 Dec 2005 01:10:37 -0000	1.157
  @@ -1222,6 +1222,8 @@
                   NodeImpl::MouseEvent mev( _mouse->stateAfter(), static_cast<NodeImpl::MouseEventType>(mouseEventType));
                   m_part->xmlDocImpl()->prepareMouseEvent( true, d->prevMouseX, d->prevMouseY, &mev );
                   oldUnder = mev.innerNode;
  +                if (oldUnder && oldUnder->isTextNode())
  +                    oldUnder = oldUnder->parentNode();
               }
               if (oldUnder != targetNode) {
                   // send mouseout event to the old node