[Webkit-unassigned] [Bug 6821] Fix for 5983 will not always update hover correctly.

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=6821


opendarwin.org at mitzpettel.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
Attachment #6113 is|0                           |1
           obsolete|                            |
   Attachment #6197|                            |review?
               Flag|                            |




------- Comment #11 from opendarwin.org at mitzpettel.com  2006-02-01 14:03 -------
Created an attachment (id=6197)
 --> (http://bugzilla.opendarwin.org/attachment.cgi?id=6197&action=view)
Updated patch

(In reply to comment #8)
> (1) You need to cancel the timer if the FrameView goes away.

Done. How come the relayout timer doesn't need to be canceled there?

> (2) You need to cancel the timer if a mouse move comes in before the timer
> fires.

Done.

> (3) Doesn't :active have all of the same issues as :hover?  It seems like the
> exact same bugs apply to :active.

It has some of the same issues (the bug 5983 ones). I added code to trim the
rendererless end of the active chain and reset m_active and m_inActiveChain,
but since the active chain is locked, I think there's no need to post a mouse
event.

ChangeLog entry and test still missing from this patch, so r- anyway.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.