1942572 - [wpt-sync] Sync PR 50171 - Resolve font-weight values with CSSLengthResolver

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, task, P4)

Description

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

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

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

Даниил <[e-mail]> wrote:

Resolve font-weight values with CSSLengthResolver

This CL adds clamping to [1, 1000] interval as per spec
https://drafts.csswg.org/css-fonts/#font-weight-prop in style builder
converter, as we now resolve calc() there as well.

This CL leaves TODO for interpolation code, as now it resolves
units before interpolation.

Bug: 40946458
Fixed: 386860174
Change-Id: I12092088aa21a001bbcd9ae4241a1dae8ca07825
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6172306
Reviewed-by: Anders Hartvoll Ruud \<[e-mail]>
Commit-Queue: Daniil Sakhapov \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1408543}

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=77e73e9ebc724c5b4e69b25732f9f50753a26377

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, and Chrome on GitHub CI

Total 1 tests and 3 subtests

Status Summary

Firefox

OK : 1
FAIL: 3

Chrome

OK : 1
FAIL: 3

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /css/css-fonts/font-weight-sign-function.html [wpt.fyi]
  • Property font-weight value 'calc(sign(1rem - 1px) * 10)': FAIL (Chrome: FAIL)
  • Property font-weight value 'calc(sign(1rem - 1px) * -10000)': FAIL (Chrome: FAIL)
  • Property font-weight value 'calc(sign(1rem - 1px) * 10000)': FAIL (Chrome: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d8cfac915011
[wpt PR 50171] - Resolve font-weight values with CSSLengthResolver, a=testonly
https://hg.mozilla.org/integration/autoland/rev/b9a506162933
[wpt PR 50171] - Update wpt metadata, a=testonly

Comment 4

[amarc]

https://hg.mozilla.org/mozilla-central/rev/d8cfac915011
https://hg.mozilla.org/mozilla-central/rev/b9a506162933