1248254 - Firefox/Android with paired/external mouse has wrong hit calculation for enter/over/leave/out

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Patrick H. Lauke]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.109 Safari/537.36

Steps to reproduce:

With Firefox on Android with a paired/external mouse, go to this basic test page http://patrickhlauke.github.io/touch/tests/event-listener.html (which outputs all sorts of events fired on the test button, including mouseover/mouseenter/mouseleave/mouseout). Move the mouse over the button.


Actual results:

There appears to be a vertical mismatch between the position of the button as seen on the screen, and the position where the mouse pointer triggers mouseover/mouseenter/mouseleave/mouseout events. This vertical mismatch seems to be exactly the size of the Firefox UI at the top of the screen - leading me to believe that Firefox is somehow getting the ABSOLUTE position of the mouse cursor on the Android screen to determine hit testing for these events, rather than compensating for the vertical offset of its web content view caused by its UI). See https://www.youtube.com/watch?v=9l8kskIUNnc

Worth noting that when actually clicking the mouse button, the hit testing seems to be happening correctly - see https://www.youtube.com/watch?v=yU8UEtzy4j0


Expected results:

mouseover/mouseenter/mouseleave/mouseout should only be fired when the actual button is hovered/unhovered with the mouse...

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

I believe Snorp said that Randal is working on this. Do we have something on file?

Comment 2

[Randall Barker [:rbarker]]

(In reply to Kevin Brosnan [:kbrosnan] from comment #1)
> I believe Snorp said that Randal is working on this. Do we have something on
> file?

Bug 1232338 - [parity] Hover from input devices does not get passed on to page CSS/javascript

I have a patch and will post it soon but it requires that Bug 1242690 land first. This is also broken in Aurora and Beta in that the Toolbar offset is not subtracted from the event. That is an easy fix if we want to uplift.

Comment 3

[Randall Barker [:rbarker]]

Attachment: 0001-Bug-1251001-Input-fields-at-the-bottom-of-a-page-do-not-pan-into-view-when-gaining-focus.-r-16022413-cb8ef3d.patch

Comment 4

[Randall Barker [:rbarker]]

Attachment: 0001-Bug-1248254-Firefox-Android-with-paired-external-mouse-has-wrong-hit-calculation-for-enter-over-leave-out-16022508-0481bde.patch

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/525acdb20f3a

Comment 6

[Randall Barker [:rbarker]]

Comment on attachment 8723619 [details] [diff] [review]
0001-Bug-1248254-Firefox-Android-with-paired-external-mouse-has-wrong-hit-calculation-for-enter-over-leave-out-16022508-0481bde.patch

Approval Request Comment
[Feature/regressing bug #]:Fix for issue cause by dynamic tool bar
[User impact if declined]:Mouse hover events in Fennec will continue to be offset by the height of the dynamic toolbar when it is visible. Things like mouse over/enter/exit etc. will be broken.
[Describe test coverage new/current, TreeHerder]: Since we don't test real mouse hover events in Fennec, there aren't any. I did test android TalkBack to make sure accessibility is not broken with this patch. 
[Risks and why]: This will only affect users with pointer devices that create hover events and does not affect touch events. This patch only adjusts the position of the pointer hover event to account for the height of the dynamic tool bar so the risk should be minimal. We currently do no support touch hover events in Fennec.
[String/UUID change made/needed]:none.

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/525acdb20f3a

Comment 8

[Randall Barker [:rbarker]]

Please note that this does not fix nightly due to APZ. This patch is intended to fix Aurora and Beta which are both still using JPZ. Bug 1232338 is required to fix this issue on nightly.

Comment 9

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8723619 [details] [diff] [review]
0001-Bug-1248254-Firefox-Android-with-paired-external-mouse-has-wrong-hit-calculation-for-enter-over-leave-out-16022508-0481bde.patch

Sorry but this is too late for 45 as we will ship in a week but taking it in 46.

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/c007817dac6c