Parse animation and transition shorthands in CSSPropertyParser with CSSParserTokens

third_party/WebKit/LayoutTests/animations/animations-parsing.html

Index: third_party/WebKit/LayoutTests/animations/animations-parsing.html
diff --git a/third_party/WebKit/LayoutTests/animations/animations-parsing.html b/third_party/WebKit/LayoutTests/animations/animations-parsing.html
index 05d7ed56d31f47ec6aef6c54e068cd4d0fa01ac4..76953290b6c2d90e18d0f431eafe11f2b180eb28 100644
--- a/third_party/WebKit/LayoutTests/animations/animations-parsing.html
+++ b/third_party/WebKit/LayoutTests/animations/animations-parsing.html
@@ -1079,6 +1079,36 @@ shouldBe("computedStyle.animation", "'none 0s ease 0s 1 normal none running'");
 shouldBe("style.webkitAnimation", "''");
 shouldBe("computedStyle.webkitAnimation", "'none 0s ease 0s 1 normal none running'");
 
+style.animation = ",";
+shouldBe("style.animation", "''");
+shouldBe("computedStyle.animation", "'none 0s ease 0s 1 normal none running'");
+shouldBe("style.webkitAnimation", "''");
+shouldBe("computedStyle.webkitAnimation", "'none 0s ease 0s 1 normal none running'");
+
+style.animation = "running,";
+shouldBe("style.animation", "''");
+shouldBe("computedStyle.animation", "'none 0s ease 0s 1 normal none running'");
+shouldBe("style.webkitAnimation", "''");
+shouldBe("computedStyle.webkitAnimation", "'none 0s ease 0s 1 normal none running'");
+
+style.animation = ", alternate";
+shouldBe("style.animation", "''");
+shouldBe("computedStyle.animation", "'none 0s ease 0s 1 normal none running'");
+shouldBe("style.webkitAnimation", "''");
+shouldBe("computedStyle.webkitAnimation", "'none 0s ease 0s 1 normal none running'");
+
+style.animation = ", commas,";
+shouldBe("style.animation", "''");
+shouldBe("computedStyle.animation", "'none 0s ease 0s 1 normal none running'");
+shouldBe("style.webkitAnimation", "''");
+shouldBe("computedStyle.webkitAnimation", "'none 0s ease 0s 1 normal none running'");
+
+style.animation = "test,, 5s";
+shouldBe("style.animation", "''");
+shouldBe("computedStyle.animation", "'none 0s ease 0s 1 normal none running'");
+shouldBe("style.webkitAnimation", "''");
+shouldBe("computedStyle.webkitAnimation", "'none 0s ease 0s 1 normal none running'");

[rwlbuis, 2015/10/20 14:32:23]

Do these extra test passes mean the behavior has improved? If so you may want to
mention it in the commit message.

[Timothy Loh, 2015/10/21 06:07:39]

It's already mentioned there. The last added test here is the only one that
currently works.

+
 // FIXME : https://code.google.com/p/chromium/issues/detail?id=273092
 /*style.animation = "ease-in ease-otu 5s";
 shouldBe("style.animation", "''");