Bug 237348

Summary:

[ iOS ] pointerevents/mouse/* 10 pointer event tests are constant timeouts or failures on iOS.

Product:

WebKit

Reporter:

Dawn Morningstar <Morningstar>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

clopez, hi, jenner, webkit-bot-watchers-bugzilla, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

See Also:

https://bugs.webkit.org/show_bug.cgi?id=233498

Attachments:

Description	Flags
expectations	none

Dawn Morningstar

Reported 2022-03-01 16:25:27 PST

pointerevents/mouse/pointer-event-basic-properties.html is a constant timeout on iOS queues ever since r287196 when it was unskipped. Current expectation is failing. HISTORY: https://results.webkit.org/?suite=layout-tests&test=pointerevents%2Fmouse%2Fpointer-event-basic-properties.html DIFF-URL: https://build.webkit.org/results/Apple-iPadOS-15-Simulator-Debug-WK2-Tests/r287196%20(555)/pointerevents/mouse/pointer-event-basic-properties-diff.txt DIFF TEXT: --- /Volumes/Data/worker/ipados-simulator-15-debug-tests-wk2/build/layout-test-results/pointerevents/mouse/pointer-event-basic-properties-expected.txt +++ /Volumes/Data/worker/ipados-simulator-15-debug-tests-wk2/build/layout-test-results/pointerevents/mouse/pointer-event-basic-properties-actual.txt @@ -1,3 +1,3 @@ +FAIL: Timed out waiting for notifyDone to be called -PASS Testing the basic properties of a pointer event triggered by a mouse.

Attachments
expectations (1.50 KB, patch) 2022-03-01 18:00 PST, Dawn Morningstar	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2022-03-01 16:29:02 PST

<rdar://problem/89648799>

Dawn Morningstar

Comment 2 2022-03-01 16:31:50 PST

Currently in r287196 all mouse functions are marked as failure on iOS with: pointerevents/mouse [ Failure ] However not all tests are designed to Fail when mouse is not detected. I believe the solution to this bug would be to update the expectation on iOS to: pointerevents/mouse [ Failure Timeout ] To account for the timeouts.

Alexey Proskuryakov

Comment 3 2022-03-01 17:14:17 PST

We don't want tests that consistently time out to run, because they are costly in terms of overall test run time. pointerevents tests are not expected to run on iOS, we should just skip (and generally, undo the effects of r287196 on iOS).

Dawn Morningstar

Comment 4 2022-03-01 17:34:13 PST

(In reply to Alexey Proskuryakov from comment #3) > We don't want tests that consistently time out to run, because they are > costly in terms of overall test run time. > > pointerevents tests are not expected to run on iOS, we should just skip (and > generally, undo the effects of r287196 on iOS). Thanks! I proposed the [ Timeout Failure ] because I assumed there MUST have been a reason for it to have been changed to a Failure expectation on iOS instead of a skip. But I agree, it would be better to skip. Fixing this bug by re-skipping on iOS.

Dawn Morningstar

Comment 5 2022-03-01 17:41:11 PST

Dawn Morningstar

Comment 6 2022-03-01 18:00:43 PST

Created attachment 453558 [details] expectations

Robert Jenner

Comment 7 2022-03-01 18:11:18 PST

Comment on attachment 453558 [details] expectations Clearing flags on attachment: 453558 Committed r290716 (247963@trunk): <https://commits.webkit.org/247963@trunk>

Note You need to log in before you can comment on or make changes to this bug.