[wpt-sync] Sync PR 50110 - Remove expected PNG files for generated OffscreenCanvas tests
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox136 fixed)
Tracking | Status | |
---|---|---|
firefox136 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 50110 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/50110
Details from upstream follow.
Jean-Philippe Gravel <jpgravel@chromium.org> wrote:
Remove expected PNG files for generated OffscreenCanvas tests
These expected images have very limited value. They are only used for
testharness tests and are not used for automatically validating the
test results. These tests are instead validated using assertions. Thus,
the expected image is only useful for manual verifications. For element
tests, test result pages include both the canvas test result and the
expected image, and thus, we can manually validate the test result
visually.For offscreen and worker tests however, the test result is never
captured from the OffscreenCanvas and never displayed in any way. It's
therefore impossible to use the expected image for verification, making
it entirely useless.Originally, the expected images almost only existed for element tests
(see the state of the tests in https://crrev.com/c/2240038 for
instance). As we merged the test definition for element and offscreen,
lots of offscreen tests mistakenly inherited the element's expected
image (see https://crrev.com/c/4562936 for instance). It's time to
clean this up.Bug: 40207206
Change-Id: I97a483ce929f7e346c8062f5ab6b6efdd6c5449d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6172745
Commit-Queue: Jean-Philippe Gravel \<jpgravel@chromium.org>
Reviewed-by: Yi Xu \<yiyix@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1406971}
Assignee | ||
Updated•1 month ago
|
Assignee | ||
Updated•29 days ago
|
Assignee | ||
Comment 1•29 days ago
|
||
Assignee | ||
Comment 3•25 days ago
|
||
Description
•