[Webkit-unassigned] [Bug 3439] mouseover effects can get stuck sometimes due to missing events

[bugzilla-daemon at opendarwin.org]

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


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #7179|review?                     |review+
               Flag|                            |




------- Comment #14 from darin at apple.com  2006-03-19 16:18 PDT -------
(From update of attachment 7179)
It's not great to pass the subframe to the passSubframeEventToSubframe
function, but on the other hand, it's my own fault for insisting the subframe
function be removed from MouseEventWithHitTestResults. Eventually we should be
able to clean that up a bit -- I'm not sure it needs to go back through the
Frame and through Macintosh-specific code.

+    RenderObject *renderer = mev.innerNode()->renderer();

New code should put the * next to the class name, not the variable.

+    RefPtr<FrameView> protector(this);

No explanation of why this was added.

I don't understand the positioning of the call to setCursor in the new code.
Wouldn't we need a new call to setCursor after passing the event to the new
subframe?

Ideally I think that oldUnder should be per-page, not per-frame. If we did
that, I think we might be able to simply get rid of oldSubframe.

Despite those doubts and comments, looks fine to land.

r=me


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