Closed Bug 1911508 Opened 3 months ago Closed 3 months ago

[wpt-sync] Sync PR 47449 - Add a WPT test validating that reset after flush discards frame content

Categories

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

task

Tracking

()

RESOLVED FIXED
131 Branch
Tracking Status
firefox131 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

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

Add a WPT test validating that reset after flush discards frame content

There was already a test validating that reset clears the frame buffer
(2d.reset.basic), but that test called reset right after the draw
command, in the same frame. Implementation might behave differently if
a frame is actually rasterized before reset is called.

In particular, Chromium has an optimization that discards pending paint
ops when reset is called. Thus, in the original test, the frame buffer
was never actually written to before the reset and the test could not
verify whether reset actually clears that buffer. By forcing frame to
be rasterized, we now validate that the frame buffer is correctly
cleared.

Bug: 354124879
Change-Id: Ia6536568e2666138ffa4a22418a7c4eb14983577
Reviewed-on: https://chromium-review.googlesource.com/5746932
WPT-Export-Revision: 2bcbb72fd90583d4e8a53647680ca48275b232fa

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

CI Results

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

Total 3 tests

Status Summary

Firefox

PASS: 3

Chrome

PASS: 3

Safari

PASS: 3

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4932017b7257 [wpt PR 47449] - Add a WPT test validating that reset after flush discards frame content, a=testonly
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 131 Branch
You need to log in before you can comment on or make changes to this bug.