Bug 121026

Summary:

MouseLeave not always emitted when cursor leaves subframe

Product:

WebKit

Reporter:

Allan Sandfeld Jensen <allan.jensen>

Component:

UI Events

Assignee:

Allan Sandfeld Jensen <allan.jensen>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

buildbot, commit-queue, esprehn+autocc, kangil.han, rniwa

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

120862

Bug Blocks:

Attachments:

Description	Flags
Patch	tonikitoo: review+, buildbot: commit-queue-
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion	none

Description Allan Sandfeld Jensen 2013-09-09 02:53:55 PDT

I noticed the recursive calls in Document::updateHoverActiveState does not forward the PlatformMouseEvent* pointer. This means the nested frames will not have enough information to emit  mouseleave.

Comment 1 Allan Sandfeld Jensen 2013-09-09 03:18:34 PDT

Created attachment 211023 [details]
Patch

Comment 2 Build Bot 2013-09-09 06:25:46 PDT

Comment on attachment 211023 [details]
Patch

Attachment 211023 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/1732374

New failing tests:
fast/events/mouseleave-on-subframe.html

Comment 3 Build Bot 2013-09-09 06:25:48 PDT

Created attachment 211034 [details]
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-04  Port: mac-mountainlion  Platform: Mac OS X 10.8.4

Comment 4 Allan Sandfeld Jensen 2013-09-09 06:47:57 PDT

 mouseenter on DIV
 mouseenter on BODY
 mouseenter on HTML
+mouseenter on #document
 mouseleave on DIV
 mouseleave on BODY
 mouseleave on HTML
+mouseleave on #document
 

Ah, perfect. Testing bug 120862

Comment 5 Allan Sandfeld Jensen 2013-09-09 06:57:18 PDT

Committed r155348: <http://trac.webkit.org/changeset/155348>