Closed Bug 1323400 Opened 3 years ago Closed 6 months ago

[PointerEvents] Long taps keeps firing the "pointermove"

Categories

(Core :: DOM: Events, defect, P3)

53 Branch
x86_64
Windows 10
defect

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox53 --- wontfix
firefox67 --- fixed

People

(Reporter: cynthiatang, Assigned: edgar)

References

(Blocks 4 open bugs)

Details

Attachments

(4 files)

Steps to reproduce:
 1. Launch Firefox with Pointer Events
 2. Open http://rbyers.github.io/eventTest.html 
 3. Long tap in an empty area

Expected result:
 - pointerType="touch"

Actual result:
 - pointerType="touch"
 - But it keeps firing the "pointermove" 
 - Videos:
    - Enable pointerEvents: https://youtu.be/a9ziGiI99E4
    - Disable pointerEvents: https://youtu.be/ms7ClnXw0Co

Environment:
 - Devices: Surface 3
 - OS: Windows 10 Pro. (64bit)
 - Firefox:53.0a1 (2016-11-30) (64-bit)
Priority: -- → P2
Without export MOZ_USE_XINPUT2=1, OS keeps firing mousemove and then we generates pointermove from those mousemove events.

As I know, the touch functionality is not working w/o export MOZ_USE_XINPUT2=1. And there are some bugs with MOZ_USE_XINPUT2=1.
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
The bug was reported on windows; however comment 1 confusingly indicated a linux configuration. Worth checking it again.
Blocks: pointerevent

I could reproduce it on Windows Surface, pointermove event keeps being fired even if the touch isn't moved.

TouchManager will filter out the duplicated events if the touch isn't changed, https://searchfox.org/mozilla-central/rev/5c8ea961d04767db723a0a15e3a8f7fbca154129/layout/base/TouchManager.cpp#276-280. And the pointer event seems doesn't handle this well.

Blocks: 1505321
Blocks: 1315250
Assignee: stone123456 → echen

To make it easier to add new test without introudcing duplicated code.

Pushed by echen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fe21a09ef43f
Part 1: Mark Touch::Equals as a const method; r=smaug
https://hg.mozilla.org/integration/autoland/rev/b526bd580215
Part 2: Filter out duplicated touchmove for pointer event; r=smaug
https://hg.mozilla.org/integration/autoland/rev/8a29492017e3
Part 3-1: Refactor test_multiple_touches.html; r=smaug
https://hg.mozilla.org/integration/autoland/rev/b49310f66efe
Part 3-2: Add test; r=smaug
Blocks: 1523998
You need to log in before you can comment on or make changes to this bug.