Closed Bug 1917097 Opened 1 year ago Closed 11 months ago

[wpt-sync] Sync PR 47996 - Moved param initialization into finalize() in the Canvas WPT generator

Categories

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

task

Tracking

()

RESOLVED FIXED
132 Branch
Tracking Status
firefox132 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Jean-Philippe Gravel <jpgravel@chromium.org> wrote:

Moved param initialization into finalize() in the Canvas WPT generator

Before this CL, the variant grid's params was initialized the first
time the grid's params property was used. This implicit
initialization was confusing because it meant we had to first use
params at the exact right time. Using it too soon was a bug (before
the variant's expected_img param is configured for instance).

This CL moves the grid's param creation into the finalize() method,
removing this confusion.

Bug: 364549423
Change-Id: I00bee16fe9f2d8ab13f5b05b49b511ffa642e746
Reviewed-on: https://chromium-review.googlesource.com/5836568
WPT-Export-Revision: 45b13c1ae7758451d59514f929780e2bc41c14fb

Component: web-platform-tests → Graphics: Canvas2D
Product: Testing → Core
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/661b2ee9b57c [wpt PR 47996] - Moved param initialization into finalize() in the Canvas WPT generator, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
You need to log in before you can comment on or make changes to this bug.