Closed Bug 1943118 Opened 1 month ago Closed 27 days ago

[wpt-sync] Sync PR 50223 - [rust png] De-flake `toBlob-cross-realm-callback-report-exception` test.

Categories

(Core :: DOM: Core & HTML, task, P4)

task

Tracking

()

RESOLVED FIXED
136 Branch
Tracking Status
firefox136 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Lukasz Anforowicz <lukasza@chromium.org> wrote:

[rust png] De-flake toBlob-cross-realm-callback-report-exception test.

Before this CL, the test assumed that frames[1].onerror will be called
within 25ms. It seems that after switching to the Rust PNG encoder this
doesn't always happen. So after this CL:

  1. The test will wait until at least one frames[i].onerror happens
    (with a timeout of 3s). This is expected to make the test more
    robust and less flaky.
  2. Only afterward the test will wait an additional 25ms to see if
    unexpectedly any other onerror has fired. This part may continue
    to be flaky, but this CL doesn't make this worse... :-/

Bug: chromium:383580201
Change-Id: Icc4c5a1bc433bb3b9401108eefe35d0dc639cd58
Reviewed-on: https://chromium-review.googlesource.com/6182295
WPT-Export-Revision: a0c854bd862fdb231acd5967cab25f164491b78b

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

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

Total 1 tests and 1 subtests

Status Summary

Firefox

OK : 1
PASS: 1

Chrome

OK : 1
FAIL: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2d0c9f8a0e36 [wpt PR 50223] - [rust png] De-flake `toBlob-cross-realm-callback-report-exception` test., a=testonly
Status: NEW → RESOLVED
Closed: 27 days ago
: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.