diff options
Diffstat (limited to '')
3 files changed, 48 insertions, 8 deletions
diff --git a/testing/web-platform/meta/css/css-transitions/CSSTransition-effect.tentative.html.ini b/testing/web-platform/meta/css/css-transitions/CSSTransition-effect.tentative.html.ini deleted file mode 100644 index 7896769374..0000000000 --- a/testing/web-platform/meta/css/css-transitions/CSSTransition-effect.tentative.html.ini +++ /dev/null @@ -1,5 +0,0 @@ -[CSSTransition-effect.tentative.html] - [After setting a transition's effect to null, a new transition can be started] - expected: - if (os == "mac") and not debug: [PASS, FAIL] - if (os == "android") and not debug: [PASS, FAIL] diff --git a/testing/web-platform/tests/css/css-animations/transition-ready-time-offscreen.html b/testing/web-platform/tests/css/css-animations/transition-ready-time-offscreen.html new file mode 100644 index 0000000000..cc3151b40b --- /dev/null +++ b/testing/web-platform/tests/css/css-animations/transition-ready-time-offscreen.html @@ -0,0 +1,44 @@ +<!doctype html> +<meta charset=utf-8> +<title>Transitions start at the same time regardless of on-screen status</title> +<link rel="help" href="https://drafts.csswg.org/css-animations-2/"> +<link rel="help" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1888748"> +<script src="/resources/testharness.js"></script> +<script src="/resources/testharnessreport.js"></script> +<style> +.transition { + width: 100px; + height: 100px; + background-color: blue; + transition: translate 10000ms linear; + will-change: translate; + translate: 0; +} +.offscreen { + background-color: purple; + translate: -1000px; +} +</style> +<div class="transition"></div> +<div class="transition offscreen"></div> +<script> +'use strict'; + +promise_test(async t => { + let [a, b] = document.querySelectorAll(".transition"); + + // Ensure a paint happens to make sure a transition gets triggered (otherwise + // getAnimations() would flush sync, and maybe not trigger a transition if + // it's the first style of the elements). + await new Promise(r => requestAnimationFrame(() => requestAnimationFrame(r))); + + // Trigger both transitions at the same time. + a.style.translate = b.style.translate = "100px"; + + let ta = a.getAnimations()[0]; + let tb = b.getAnimations()[0]; + + await Promise.all([ta.ready, tb.ready]); + assert_equals(ta.startTime, tb.startTime, "Both transitions should've started at the same time"); +}); +</script> diff --git a/testing/web-platform/tests/css/css-transitions/CSSTransition-effect.tentative.html b/testing/web-platform/tests/css/css-transitions/CSSTransition-effect.tentative.html index b58c93d2e6..0952a7cd8f 100644 --- a/testing/web-platform/tests/css/css-transitions/CSSTransition-effect.tentative.html +++ b/testing/web-platform/tests/css/css-transitions/CSSTransition-effect.tentative.html @@ -85,12 +85,13 @@ promise_test(async t => { div.style.left = '150px'; // This will run style update. - assert_equals(div.getAnimations().length, 1); + const animations = div.getAnimations(); + assert_equals(animations.length, 1); - const new_transition = div.getAnimations()[0]; + const new_transition = animations[0]; await new_transition.ready; - assert_equals(getComputedStyle(div).left, '100px'); + assert_not_equals(getComputedStyle(div).left, '150px'); }, 'After setting a transition\'s effect to null, a new transition can be started'); // This is a regression test for https://crbug.com/992668, where Chromium would |