1962018 - [wpt-sync] Sync PR 52109 - Disallow calc() in named timeline range keyframe selectors

Status: RESOLVED FIXED

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

Description

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

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

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

Rune Lillesveen <[e-mail]> wrote:

Disallow calc() in named timeline range keyframe selectors

Pure percentage selectors already disallowed calc() expressions. The
named selectors should not be different. It is not obvious from the spec
that this should be the behavior, but having selectors with calc() that
contains relative lengths or sibling-index() would have to be dropped
regardlessly.

Bug: 410883045
Change-Id: I56e0c57f9cdaae9490b47844dad9e06fde09c214
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6476968
Commit-Queue: Rune Lillesveen \<[e-mail]>
Reviewed-by: Daniil Sakhapov \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1450184}

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=8be6ea36ad42da3348b5f6612288b3276256473a

Comment 2

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

CI Results

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

Total 1 tests and 14 subtests

Status Summary

Firefox

OK : 1
PASS: 12
FAIL: 2

Chrome

OK : 1
PASS: 12
FAIL: 2

Safari

OK : 1
PASS: 10
FAIL: 4

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

• /css/css-animations/parsing/keyframe-selectors.html [wpt.fyi]
  • 'entry 10%' should be a valid keyframe selector: FAIL
  • 'exit 60%' should be a valid keyframe selector: FAIL

New Tests That Don't Pass

• /css/css-animations/parsing/keyframe-selectors.html [wpt.fyi]
  • 'entry 10%' should be a valid keyframe selector: FAIL (Chrome: PASS, Safari: PASS)
  • 'exit 60%' should be a valid keyframe selector: FAIL (Chrome: PASS, Safari: PASS)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/c58e9eec72fa
[wpt PR 52109] - Disallow calc() in named timeline range keyframe selectors, a=testonly
https://hg.mozilla.org/mozilla-central/rev/8e86d3ee9975
[wpt PR 52109] - Update wpt metadata, a=testonly

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/c58e9eec72fa
https://hg.mozilla.org/mozilla-central/rev/8e86d3ee9975