Consider to make *enter/*leave events uncomposed

RESOLVED FIXED in Firefox 63

Status

()

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: smaug, Assigned: smaug)

Tracking

unspecified
mozilla63
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(3 attachments)

https://github.com/w3c/uievents/issues/208#issuecomment-413963206 sounds like at least Apple would be ok with this. Trying to get feedback from Google and MS too.
Based on the github issue, sound like people in general are in favor of the change. I'd rather fix our behavior sooner than later, so that pages like ea.com works with shadow DOM enabled FF.

The test is very simple but enough to cover the change.

remote: Follow the progress of your build on Treeherder:
remote:   https://treeherder.mozilla.org/#/jobs?repo=try&revision=a4e3946334ea25b05ef1bda76b091506e5f0ff1f
remote: recorded changegroup in replication log in 0.011s
Assignee: nobody → bugs
Attachment #9002926 - Flags: review?(masayuki)
Comment on attachment 9002926 [details] [diff] [review]
composed_enterleave.diff

Review of attachment 9002926 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/events/test/file_bug1484371.html
@@ +24,5 @@
> +        return "onpointerenter" in document.body;
> +      }
> +
> +      function checkEventCounts(expected) {
> +        parent.is(mouseEnterCount, expected.mouseEnterCount, "mouseenter event count");

Please make checkEventCounts take prefix string for the parent.is(). E.g., I like "moving on lightDiv: mouseenter event count" instead of jus t"mouseenter event count".

::: dom/events/test/mochitest.ini
@@ +205,5 @@
>  [test_bug1447993.html]
>  support-files = window_bug1447993.html
>  skip-if = toolkit == 'android'
> +[test_bug1484371.html]
> +support-files = file_bug1484371.html

Hmm, I like test_mouse_enter_leave_event_count_on_shadow_host.html or something to explain what this check since this is not Gecko's bug from a point of view of future developers, just conforming to (new) spec. But up to you.
Attachment #9002926 - Flags: review?(masayuki) → review+
Posted patch untestedSplinter Review
(would need to update everything on this older laptop to be able to compile Nightly. Will test and land in a moment using another laptop)
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/971943621f6e
make *enter/*leave events uncomposed, r=masayuki
https://hg.mozilla.org/mozilla-central/rev/971943621f6e
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.