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

bugzilla-daemon at opendarwin.org bugzilla-daemon at opendarwin.org
Wed Feb 1 14:03:49 PST 2006


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.


> (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

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.

More information about the webkit-unassigned mailing list