1911815 - [wpt-sync] Sync PR 47481 - Scheduling APIs: Remove scheduler.yield() parameters

Status: RESOLVED FIXED

(Core :: DOM: Performance APIs, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

Scott Haseley <[e-mail]> wrote:

Scheduling APIs: Remove scheduler.yield() parameters

By default, scheduler.yield() continuations inherit the priority and
abort signal from the current task, but this behavior can be overridden
by providing a signal or priority option. But it's not clear if this is
necessary, and there are questions around the expectations of subsequent
continuations (see https://github.com/WICG/scheduling-apis/issues/96).

So to simplify the API and prevent shipping behavior we might regret,
we're removing the yield() parameters. If control over a continuation's
priority/abort is needed, a new task can be scheduled using postTask().
We plan to reevaluate this as the API rolls out and developers start
using it in more complex ways.

Related PR: https://github.com/WICG/scheduling-apis/pull/100.

Bug: 40633887
Change-Id: I5e23c9c8cf950cec795cddafd27c962b71eea5f7

Reviewed-on: https://chromium-review.googlesource.com/5753325
WPT-Export-Revision: 08eac15b478b582e857857427a9983095725f37b

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=5b57393fc4b339142bd105c04f2032bc848d64de

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

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

Total 9 tests and 1 subtests

Status Summary

Firefox

OK : 5
PASS : 2
FAIL : 18
TIMEOUT: 4
ERROR : 3

Chrome

OK : 9
PASS : 23

Safari

OK : 4
FAIL : 14
TIMEOUT: 4
ERROR : 4

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

• /scheduler/tentative/yield/yield-abort.any.html [wpt.fyi]: ERROR

New Tests That Don't Pass

• /scheduler/tentative/yield/yield-abort.any.worker.html [wpt.fyi]
  • yield() aborted in a separate task: FAIL (Chrome: PASS, Safari: FAIL)
• /scheduler/tentative/yield/yield-abort.any.html [wpt.fyi]: ERROR (Chrome: OK, Safari: OK)
  • yield() aborted in a separate task: FAIL (Chrome: PASS, Safari: FAIL)
• /scheduler/tentative/yield/yield-inherit-across-promises.any.html [wpt.fyi]
  • yield() inherits priority (string) across promises (user-blocking): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits priority (signal) across promises (user-blocking): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits priority (string) across promises (background): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits priority (signal) across promises (background): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits abort across promises: FAIL (Chrome: PASS, Safari: FAIL)
• /scheduler/tentative/yield/yield-inherit-across-promises.any.worker.html [wpt.fyi]
  • yield() inherits priority (string) across promises (user-blocking): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits priority (signal) across promises (user-blocking): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits priority (string) across promises (background): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits priority (signal) across promises (background): FAIL (Chrome: PASS, Safari: FAIL)
  • yield() inherits abort across promises: FAIL (Chrome: PASS, Safari: FAIL)
• /scheduler/tentative/yield/yield-priority-idle-callbacks.html [wpt.fyi]: ERROR (Chrome: OK, Safari: ERROR)
  • requestIdleCallback() yields at background priority when inheriting signal: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /scheduler/tentative/yield/yield-priority-posttask.any.worker.html [wpt.fyi]
  • yield() with postTask tasks (priority): FAIL (Chrome: PASS)
  • yield() with postTask tasks (signal): FAIL (Chrome: PASS)
  • yield() with TaskSignal has dynamic priority: FAIL (Chrome: PASS)
• /scheduler/tentative/yield/yield-priority-posttask.any.html [wpt.fyi]
  • yield() with postTask tasks (priority): FAIL (Chrome: PASS)
  • yield() with postTask tasks (signal): FAIL (Chrome: PASS)
  • yield() with TaskSignal has dynamic priority: FAIL (Chrome: PASS)
• /scheduler/tentative/yield/yield-priority-timers.any.html [wpt.fyi]: ERROR (Chrome: OK, Safari: ERROR)
  • yield() with timer tasks (inherit signal): TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /scheduler/tentative/yield/yield-priority-timers.any.worker.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • yield() with timer tasks (inherit signal): TIMEOUT (Chrome: PASS, Safari: TIMEOUT)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/92c0f0d7cc12
[wpt PR 47481] - Scheduling APIs: Remove scheduler.yield() parameters, a=testonly
https://hg.mozilla.org/integration/autoland/rev/63982c6439b1
[wpt PR 47481] - Update wpt metadata, a=testonly

Comment 4

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/92c0f0d7cc12
https://hg.mozilla.org/mozilla-central/rev/63982c6439b1