Bug 156971 – Mouseleave fires on elements no longer in the DOM

• Component: DOM
• Status: RESOLVED CONFIGURATION CHANGED

[Becky Conning]

Steps:
1. Access http://jsfiddle.net/cuiffo/czmohkvo/2/
2. Move mouse over the black box
3. Click the black box
4. Move the mouse after waiting
5. Expect no "Mouse leave fired on inner" messages to be presented.

Actual results:
On Safari "Mouse leave fired on inner" appears because mouseleave is fired.

Expected results:
On Firefox, Chrome, Opera and Internet Explorer it does not.

Build Date & Platform:
Version 9.1.1 (11601.6.15, 11602.1.27)

See https://bugs.chromium.org/p/chromium/issues/detail?id=515921 for the corresponding WebKit bug that was fixed late last year.

[Becky Conning]

Correction:

See https://bugs.chromium.org/p/chromium/issues/detail?id=515921 for the corresponding Chromium bug that was fixed late last year.

[Radar WebKit Bug Importer]

<rdar://problem/25906098>

[David Kilzer (:ddkilzer)]

(In reply to comment #1)
> Correction:
> 
> See https://bugs.chromium.org/p/chromium/issues/detail?id=515921 for the
> corresponding Chromium bug that was fixed late last year.

Blink commit where this is fixed:

<https://src.chromium.org/viewvc/blink?view=revision&revision=200656>

[David Kilzer (:ddkilzer)]

(In reply to comment #3)
> (In reply to comment #1)
> > Correction:
> > 
> > See https://bugs.chromium.org/p/chromium/issues/detail?id=515921 for the
> > corresponding Chromium bug that was fixed late last year.
> 
> Blink commit where this is fixed:
> 
> <https://src.chromium.org/viewvc/blink?view=revision&revision=200656>

And the upstream commit log:

'''
Skipping mouseenter/over/out/leave on deleted nodes.

Both IE and FF skip firing of mouse transition events at deleted nodes,
with only IE doing it in a perfect manner by maintining the pairing of
entry/exit events. A crack in our code had been causing firing of these
events at a node after the node got deleted. This CL fixes the bug by
making chromium behave similar to FF. The perfect solution seems
non-trivial.

BUG=515921

Review URL: https://codereview.chromium.org/1288483003
'''

[David Kilzer (:ddkilzer)]

Created attachment 277398 [details]
Quick port of Blink patch (not for landing; not vetted)

[David Kilzer (:ddkilzer)]

Created attachment 277408 [details]
Quick port of Blink patch v2 (not for landing; not vetted)

[Build Bot]

Comment on attachment 277408 [details]
Quick port of Blink patch v2 (not for landing; not vetted)

Attachment 277408 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1225459

New failing tests:
fast/events/mouse-events-on-node-deletion.html

[Build Bot]

Created attachment 277412 [details]
Archive of layout-test-results from ews100 for mac-yosemite

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

[Build Bot]

Comment on attachment 277408 [details]
Quick port of Blink patch v2 (not for landing; not vetted)

Attachment 277408 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/1225460

New failing tests:
fast/events/mouse-events-on-node-deletion.html

[Build Bot]

Created attachment 277414 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.4

[Build Bot]

Comment on attachment 277408 [details]
Quick port of Blink patch v2 (not for landing; not vetted)

Attachment 277408 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/1225521

New failing tests:
fast/events/mouse-events-on-node-deletion.html

[Build Bot]

Created attachment 277417 [details]
Archive of layout-test-results from ews112 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-yosemite  Platform: Mac OS X 10.10.5

[Lucas Forschler]

Mass move bugs into the DOM component.

[Ryosuke Niwa]

*** Bug 186094 has been marked as a duplicate of this bug. ***

[Ahmad Saleem]

I am able to reproduce this bug on Safari Technology Preview 182 as well. Firefox Nightly 121 and Chrome Canary 121 match other and don't fire event after mouse leave container in test case (post clicking - delete).

[Alexsander Borges Damaceno]

Pull request: https://github.com/WebKit/WebKit/pull/30098

[Alexsander Borges Damaceno]

Pull request: https://github.com/WebKit/WebKit/pull/30099

[Ryosuke Niwa]

I can't reproduce this problem in STP 214.

[Ryosuke Niwa]

mouseleave doesn't get dispatched in the original test case written in jsfiddle.