1877447 - Perma /pointerevents/pointerevent_pointer_boundary_events_after_removing_last_over_element.html | After removing the last over element, redundant pointerenter events should not be fired on the ancestors | when Gecko 124 switches to late Beta on 2024-03-01

Status: VERIFIED FIXED

(Core :: DOM: Events, defect)

Description

[pstanciu]

Central-as-late-beta-simulation
How to run these simulations
Failure log

[task 2024-01-30T11:57:49.542Z] 11:57:49     INFO - TEST-START | /pointerevents/pointerevent_pointer_boundary_events_after_removing_last_over_element.html
[task 2024-01-30T11:57:49.542Z] 11:57:49     INFO - Setting pref dom.w3c_pointer_events.getcoalescedevents_only_in_securecontext to false
[task 2024-01-30T11:57:49.575Z] 11:57:49     INFO - Setting pref dom.w3c_pointer_events.getcoalescedevents_only_in_securecontext to true
[task 2024-01-30T11:57:49.604Z] 11:57:49     INFO - Setting pref layout.reflow.synthMouseMove to true
[task 2024-01-30T11:57:49.620Z] 11:57:49     INFO - Closing window a394d45b-2748-4d21-bc99-b06a8054fb92
[task 2024-01-30T11:57:50.126Z] 11:57:50     INFO - {'actions': [{'type': 'none', 'actions': [{'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}], 'id': '0'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 18, 'y': 50, 'origin': 'viewport'}, {'type': 'pointerDown', 'button': 0}, {'type': 'pointerUp', 'button': 0}, {'type': 'pointerMove', 'x': 18, 'y': 18, 'origin': 'viewport'}], 'parameters': {'pointerType': 'mouse'}, 'id': '1'}]}
[task 2024-01-30T11:57:50.210Z] 11:57:50     INFO - .
[task 2024-01-30T11:57:50.210Z] 11:57:50     INFO - TEST-OK | /html/browsers/origin/cross-origin-objects/location-properties-smoke-test.window.html?valueOf | took 785ms
[task 2024-01-30T11:57:50.213Z] 11:57:50     INFO - TEST-START | /html/browsers/origin/cross-origin-objects/location-properties-smoke-test.window.html?toString
[task 2024-01-30T11:57:50.218Z] 11:57:50     INFO - Closing window db4aee97-8973-4521-b7a4-883040a8ef12
[task 2024-01-30T11:57:50.272Z] 11:57:50     INFO - {'actions': [{'type': 'none', 'actions': [{'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}], 'id': '2'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 18, 'y': 114, 'origin': 'viewport'}, {'type': 'pointerDown', 'button': 0}, {'type': 'pointerUp', 'button': 0}, {'type': 'pointerMove', 'x': 18, 'y': 18, 'origin': 'viewport'}], 'parameters': {'pointerType': 'mouse'}, 'id': '3'}]}
[task 2024-01-30T11:57:50.403Z] 11:57:50     INFO - 
[task 2024-01-30T11:57:50.403Z] 11:57:50     INFO - TEST-UNEXPECTED-FAIL | /pointerevents/pointerevent_pointer_boundary_events_after_removing_last_over_element.html | After removing the last over element, redundant pointerenter events should not be fired on the ancestors - assert_equals: expected "pointerleave@div#parent, pointerleave@div#grandparent, pointerover@body, pointermove@body" but got "pointerover@body, pointerenter@body, pointermove@body"
[task 2024-01-30T11:57:50.403Z] 11:57:50     INFO - @http://web-platform.test:8000/pointerevents/pointerevent_pointer_boundary_events_after_removing_last_over_element.html:87:18
[task 2024-01-30T11:57:50.403Z] 11:57:50     INFO - 
[task 2024-01-30T11:57:50.403Z] 11:57:50     INFO - TEST-UNEXPECTED-FAIL | /pointerevents/pointerevent_pointer_boundary_events_after_removing_last_over_element.html | After removing the root element in the shadow under the cursor, pointerleave events should be targeted outside the shadow, but redundant pointerenter events should not be fired - assert_equals: expected "pointerleave@div#shadowHost, pointerleave@div#containerOfShadowHost, pointerover@body, pointermove@body" but got "pointerover@body, pointerenter@body, pointermove@body"
[task 2024-01-30T11:57:50.403Z] 11:57:50     INFO - @http://web-platform.test:8000/pointerevents/pointerevent_pointer_boundary_events_after_removing_last_over_element.html:139:18
[task 2024-01-30T11:57:50.403Z] 11:57:50     INFO - TEST-OK | /pointerevents/pointerevent_pointer_boundary_events_after_removing_last_over_element.html | took 861ms

Comment 1

[pstanciu]

The following test is also failing:

• /uievents/mouse/mouse_boundary_events_after_removing_last_over_element.html | After removing the last over element, redundant mouseenter events should not be fired on the ancestors - assert_equals: expected "mouseover@div#parent, mouseout@div#parent, mouseleave@div#parent, mouseleave@div#grandparent, mouseover@body, mousemove@body" but got "mouseover@div#parent, mouseenter@body, mouseenter@div#grandparent, mouseenter@div#
  Link to log

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Odd, I added .ini files for the known failures, but they have been removed accidentally before landing them. I'll fix it today.

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Err, this patch has the fix, but the file name is old ones (containing .tentative).

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1877447 - Take the `.ini` files back r=smaug!

I added the .ini files with proper content in D199469, but their file names
are old ones (containing .tentative) and although I don't know the reasons,
they are not in the tree. Therefore, I just copy the content into the .ini
files.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1f20700cb740
Take the `.ini` files back r=smaug

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/1f20700cb740

Comment 7

[Cristian Tuns]

The patch is working in today's Late beta-simulation: https://treeherder.mozilla.org/jobs?repo=try&group_state=expanded&revision=6624d1035899bae0df2baa2ac689b6e7972d87c3&searchStr=linux%2C18.04%2Cx64%2Cwebrender%2Cdebug%2Cweb%2Cplatform%2Ctests%2Cwith%2Csoftware%2Cwebrender%2Cenabled%2Cwithout%2Cfission%2Cenabled%2Ctest-linux1804-64-qr%2Fdebug-web-platform-tests-swr-nofis%2Cwpt