Closed Bug 1658004 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 24923 - [EventTiming] Improve some of the flaky tests

Categories

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

task

Tracking

(firefox82 fixed)

RESOLVED FIXED
82 Branch
Tracking Status
firefox82 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Nicolás Peña Moreno <npm@chromium.org> wrote:

[EventTiming] Improve some of the flaky tests

This CL improves crossiframe.html by waiting for the load event (so it
knows the iframe is ready), and makes it faster by awaiting for both the
main frame and iframe observers at the same time. It also improves the
buffered-and-duration test by considering cases in which the first event
does actually get buffered due to machine slowness.

Bug: 1074048
Change-Id: Ieb356a4a4bbcfca8454e8a76334df70867b4f954
Reviewed-on: https://chromium-review.googlesource.com/2343487
WPT-Export-Revision: 12cf46513a14dc8675ce706b59c7fbfe3bc1a309

CI Results

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

Total 2 tests and 1 subtests

Status Summary

Firefox

OK : 2
FAIL: 2

Chrome

OK : 2
PASS: 1
FAIL: 1

Safari

OK : 2
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/event-timing/buffered-and-duration-threshold.html
PerformanceObserver buffering independent of durationThreshold: FAIL (Chrome: PASS, Safari: FAIL)
/event-timing/crossiframe.html
Event Timing: entries should only be observable by its own frame.: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b4da75e7e1fd
[wpt PR 24923] - [EventTiming] Improve some of the flaky tests, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.