Skip to content

Commit

Permalink
Update style before computing current animations
Browse files Browse the repository at this point in the history
This allows CSS animations to be in a proper state for the call.

Bug: 813908
Change-Id: Ibd3870432b0c6fc8cc1766fe5c8abec3263db879
  • Loading branch information
stephenmcgruer authored and chromium-wpt-export-bot committed Feb 23, 2018
1 parent aa7041b commit e79fef0
Showing 1 changed file with 34 additions and 0 deletions.
34 changes: 34 additions & 0 deletions css/css-animations/pending-style-changes-001.html
@@ -0,0 +1,34 @@
<meta charset=utf-8>
<title>CSS Animations Test: requirement on pending style changes - getAnimations</title>
<link rel="help" href="https://drafts.csswg.org/css-animations-2/#requirements-on-pending-style-changes">

<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>

<style>
@keyframes anim {}

.animate {
animation: anim 10s;
}
</style>

<div id="target"></div>

<script>
test (t => {
assert_equals(target.getAnimations().length, 0, 'Test precondition.');
target.classList.add('animate');
assert_equals(target.getAnimations().length, 1,
'target.getAnimations() should include the CSS animation after animate class added.');
target.classList.remove('animate');
}, 'Animatable::getAnimations() should be able to see a style-created CSS animation immediately');

test(t => {
assert_equals(document.getAnimations().length, 0, 'Test precondition.');
target.classList.add('animate');
assert_equals(document.getAnimations().length, 1,
'document.getAnimations() should include the CSS animation after animate class added.');
target.classList.remove('animate');
}, 'Document::getAnimations() should be able to see a style-created CSS animation immediately');
</script>

0 comments on commit e79fef0

Please sign in to comment.