Closed Bug 1941942 Opened 1 month ago Closed 25 days ago

[wpt-sync] Sync PR 50110 - Remove expected PNG files for generated OffscreenCanvas tests

Categories

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

task

Tracking

(firefox136 fixed)

RESOLVED FIXED
136 Branch
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}

Component: web-platform-tests → Graphics: Canvas2D
Product: Testing → Core
Component: Graphics: Canvas2D → web-platform-tests
Product: Core → Testing
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6485f10d83b5 [wpt PR 50110] - Remove expected PNG files for generated OffscreenCanvas tests, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 25 days ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.