Closed Bug 1613273 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 21587 - Add touch-action:none to multi touch tests to prevent firing gestures.

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox75 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla75
Tracking Flags:
Tracking Status
firefox75 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 21587 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/21587
Details from upstream follow.

Luis Sanchez Padilla <lusanpad@microsoft.com> wrote:

Add touch-action:none to multi touch tests to prevent firing gestures.

These tests might produce gestures due to the pointer moves that are
being fired, but they are not trying to exercise such behavior. They
currently assume a slop region of 15 pixels as a requirement for
gestures to get fired, but if that region shrinks, these tests will
fail. Adding touch-action:none in the target element's style ensures
that no gesture gets produced.

Bug: 1048893
Change-Id: Iabe6c1dbc9a710a5417aa38290ccf37fd958f198

Reviewed-on: https://chromium-review.googlesource.com/2037960
WPT-Export-Revision: b79862052051430e3134f1cef59e33e3ba2255eb

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

CI Results

Ran 13 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 48 tests

Status Summary

Firefox

PASS : 43
ERROR: 5

Chrome

OK : 5
PASS : 5

Safari

ERROR: 5

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/infrastructure/testdriver/actions/multiTouchPointsReleaseFirstPoint.html: ERROR (Chrome: OK, Safari: ERROR)
/infrastructure/testdriver/actions/multiTouchPoints.html: ERROR (Chrome: OK, Safari: ERROR)
/infrastructure/testdriver/actions/multiTouchPointsTwoTouchStarts.html: ERROR (Chrome: OK, Safari: ERROR)
/infrastructure/testdriver/actions/multiTouchPointsReleaseSecondPoint.html: ERROR (Chrome: OK, Safari: ERROR)
/infrastructure/testdriver/actions/multiTouchPointsWithPause.html: ERROR (Chrome: OK, Safari: ERROR)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1d381eeb247a [wpt PR 21587] - Add touch-action:none to multi touch tests to prevent firing gestures., a=testonly

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

Status: REOPENED → RESOLVED
Closed: 5 years ago → 5 years ago
status-firefox75: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.