Closed Bug 1699420 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 28130 - Deflake svg/animations/animate-reset-freeze.html

Categories

(Core :: SVG, task, P4)

task

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Fredrik S\xc3\xb6derqvist <fs@opera.com>' wrote:

Deflake svg/animations/animate-reset-freeze.html

The timer used in this test could fire before DOMContentLoaded (and
'load') was dispatched and thus end up with the js-test default logging
as well as the PASS/FAIL. This would also have meant that the test may
not have been testing actually rewinding the timeline (resetting
animations) since animations won't be applied until after 'load' has
been dispatched. Add an additional assertion to make sure the animation
was applied at t=100.

While here, convert the test to use testharness.js and move it to WPT.

Change-Id: I525b2deda79ce71f5f18264bda5adc86c10c0139
Reviewed-on: https://chromium-review.googlesource.com/2772423
WPT-Export-Revision: dca18d2270d85896d29166bd8a53c71e545d3e12

Component: web-platform-tests → SVG
Product: Testing → Core

CI Results

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

Total 1 tests and 1 subtests

Status Summary

Firefox

OK : 1
PASS: 1

Chrome

OK : 1
PASS: 1

Safari

OK : 1
PASS: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/522ba5902102
[wpt PR 28130] - Deflake svg/animations/animate-reset-freeze.html, a=testonly
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in before you can comment on or make changes to this bug.