Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(471)

Unified Diff: third_party/WebKit/LayoutTests/animations/keyword-composition-crash.html

Issue 1720403002: Alternative syntax for element.animate list of keyframes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@animations-keyframeeffect-api
Patch Set: Change to error types and how offset/composite are handled, plus minor fixes Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/animations/keyword-composition-crash.html
diff --git a/third_party/WebKit/LayoutTests/animations/keyword-composition-crash.html b/third_party/WebKit/LayoutTests/animations/keyword-composition-crash.html
index 2611fdb89dad6f262238be05a307253dc7116060..5b93f9ef1636b9bd921bf1de89dfa0928eaf03b8 100644
--- a/third_party/WebKit/LayoutTests/animations/keyword-composition-crash.html
+++ b/third_party/WebKit/LayoutTests/animations/keyword-composition-crash.html
@@ -3,8 +3,8 @@
<div id="target"></div>
<script>
test(() => {
- target.animate({position: 'absolute'}, {fill: 'forwards'});
- target.animate({position: 'static', composite: 'add'}, {fill: 'forwards'});
+ target.animate([{position: 'absolute'}], {fill: 'forwards'});
+ target.animate([{position: 'static', composite: 'add'}], {fill: 'forwards'});
getComputedStyle(target).position;
}, "Don't crash when compositing keyword animations.");
</script>

Powered by Google App Engine
This is Rietveld 408576698