Mouseleave fires on elements no longer in the DOM
https://bugs.webkit.org/show_bug.cgi?id=156971
Summary Mouseleave fires on elements no longer in the DOM
Becky Conning
Reported 2016-04-25 00:14:36 PDT
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.
Attachments
Quick port of Blink patch (not for landing; not vetted) (15.82 KB, patch)
2016-04-26 10:35 PDT, David Kilzer (:ddkilzer)
no flags
Quick port of Blink patch v2 (not for landing; not vetted) (15.85 KB, patch)
2016-04-26 14:45 PDT, David Kilzer (:ddkilzer)
buildbot: commit-queue-
Archive of layout-test-results from ews100 for mac-yosemite (776.95 KB, application/zip)
2016-04-26 15:29 PDT, Build Bot
no flags
Archive of layout-test-results from ews123 for ios-simulator-wk2 (636.82 KB, application/zip)
2016-04-26 15:37 PDT, Build Bot
no flags
Archive of layout-test-results from ews112 for mac-yosemite (853.74 KB, application/zip)
2016-04-26 15:57 PDT, Build Bot
no flags
Becky Conning
Comment 1 2016-04-25 01:24:00 PDT
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
Comment 2 2016-04-25 07:23:00 PDT
David Kilzer (:ddkilzer)
Comment 3 2016-04-25 12:05:10 PDT
(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)
Comment 4 2016-04-25 12:06:44 PDT
(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)
Comment 5 2016-04-26 10:35:48 PDT
Created attachment 277398 [details] Quick port of Blink patch (not for landing; not vetted)
David Kilzer (:ddkilzer)
Comment 6 2016-04-26 14:45:08 PDT
Created attachment 277408 [details] Quick port of Blink patch v2 (not for landing; not vetted)
Build Bot
Comment 7 2016-04-26 15:29:38 PDT
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
Comment 8 2016-04-26 15:29:43 PDT
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 9 2016-04-26 15:37:51 PDT
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
Comment 10 2016-04-26 15:37:55 PDT
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 11 2016-04-26 15:57:00 PDT
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
Comment 12 2016-04-26 15:57:04 PDT
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
Comment 13 2019-02-06 09:19:14 PST
Mass move bugs into the DOM component.
Ryosuke Niwa
Comment 14 2019-02-06 21:05:32 PST
*** Bug 186094 has been marked as a duplicate of this bug. ***
Ahmad Saleem
Comment 15 2023-11-02 02:24:44 PDT
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
Comment 16 2024-06-23 14:17:33 PDT
Alexsander Borges Damaceno
Comment 17 2024-06-23 14:34:22 PDT
Note You need to log in before you can comment on or make changes to this bug.