Closed Bug 1904983 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 46919 - Update the canvas state and the FontDescription to match

Categories

(Core :: Graphics: Canvas2D, task, P4)

task

Tracking

()

RESOLVED FIXED
130 Branch
Tracking Status
firefox130 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Regressed 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

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

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

Yi Xu <yiyix@chromium.org> wrote:

Update the canvas state and the FontDescription to match

After a font is set in Canvas, users can change the font properties,
such as FontKerning. One copy of the properties is saved in the
FontDescription and the another copy is saved in the
CanvasRenderingContext2DState. When font is updated, the copy in the
FontDescription get updated automatically, and the copy in the State
file stay unchanged. This inconsistency caused some properties
settings to fail.

Bug: 338965374

Change-Id: Ic2e72f8d1a5e5007648dfa7da7698d836ef54c42
Reviewed-on: https://chromium-review.googlesource.com/5637432
WPT-Export-Revision: 480db9dd742c84b4989ce3e0c343884d5d748a8e

Component: web-platform-tests → Graphics: Canvas2D
Product: Testing → Core

CI Results

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

Total 15 tests and 1 subtests

Status Summary

Firefox

OK : 9
PASS: 13
FAIL: 2

Chrome

OK : 9
PASS: 9
FAIL: 6

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/aa80dcf3f20a [wpt PR 46919] - Update the canvas state and the FontDescription to match, a=testonly https://hg.mozilla.org/integration/autoland/rev/86125713b662 [wpt PR 46919] - Update wpt metadata, a=testonly
Regressions: 1906880
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 130 Branch
You need to log in before you can comment on or make changes to this bug.