[wpt-sync] Sync PR 59314 - Extend context loss/restoration test coverage to desynchronized canvas
Categories
(Core :: Graphics: Canvas2D, task, P4)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox152 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 59314 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/59314
Details from upstream follow.
Jean-Philippe Gravel <jpgravel@chromium.org> wrote:
Extend context loss/restoration test coverage to desynchronized canvas
Desynchronize canvases use a resource dispatcher to push frames and
since the dispatcher is descarded on context loss [1], it's worth adding
test coverage for this.Note that the frame dispatcher is never recreated after the context is
restored, so the canvas doesn't fully recover from the context loss. The
canvas can be read back though, so the tests still works and we can at
least validate that the context state transitions work properly.Also note that the OffscreenCanvas doesn't do much with the
desynchronizedflag, but the test coverage is still useful given that
this flag is listed as supported in MDN [1] and the specification [2].[1] https://crsrc.org/c/third_party/blink/renderer/modules/canvas/canvas2d/canvas_rendering_context_2d.cc;l=290;drc=8abea14deda089834ba142a35e8342014812df55
[2] https://developer.mozilla.org/en-US/docs/Web/API/OffscreenCanvas/getContext
[3] https://html.spec.whatwg.org/multipage/canvas.html#canvas-setting-init-bitmapChange-Id: I300d1f2827406120896cc7dc38794cb450ea3ed5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7770647
Reviewed-by: Vasiliy Telezhnikov \<vasilyt@chromium.org>
Commit-Queue: Jean-Philippe Gravel \<jpgravel@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1617254}
| Assignee | ||
Updated•1 month ago
|
| Assignee | ||
Comment 1•1 month ago
|
||
| Assignee | ||
Comment 2•1 month ago
|
||
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
| Assignee | ||
Comment 4•1 month ago
|
||
Test result changes from PR not available.
| Assignee | ||
Comment 6•1 month ago
|
||
Test result changes from PR not available.
| Assignee | ||
Comment 7•1 month ago
|
||
Test result changes from PR not available.
| Assignee | ||
Comment 8•29 days ago
|
||
Test result changes from PR not available.
Description
•