Don't check for property tree nodes for ignored transform animations.
Transform properties are ignored on inline elements. This code change
makes us stop expecting to have a property tree node for an animation
that is animating a transform property on an inline element. This means
that we won't assert about the inconsistency of having some but not all
of the expected property tree nodes, which was introduced in
https://crrev.com/c/3307117 , when we stopped creating all such nodes
for any animation on the compositor.
We still don't animate such animations on the compositor, but this does
add a comment noting that we might be able to do so, and probably means
there's one less stumbling block in the way of doing so.
The added test hits a DCHECK() without the code change.
Fixed: 1316688
Change-Id: Id2ab358261467ecc716af586df0b9d8e9baff200
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3601169
Reviewed-by: Robert Flack <flackr@chromium.org>
Commit-Queue: David Baron <dbaron@chromium.org>
Cr-Commit-Position: refs/heads/main@{#995344}
diff --git a/web-animations/animation-model/keyframe-effects/transform-and-opacity-on-inline-001-ref.html b/web-animations/animation-model/keyframe-effects/transform-and-opacity-on-inline-001-ref.html
new file mode 100644
index 0000000..1e7f250
--- /dev/null
+++ b/web-animations/animation-model/keyframe-effects/transform-and-opacity-on-inline-001-ref.html
@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<html>
+<title>CSS Test (Animations): Element.animate() animating both transform and opacity on an inline</title>
+<link rel="author" title="L. David Baron" href="https://dbaron.org/">
+<link rel="author" title="Google" href="http://www.google.com/">
+
+<style>
+#target {
+ opacity: 0.4;
+ will-change: opacity;
+}
+</style>
+
+<body><span id="target">x</span></body>
diff --git a/web-animations/animation-model/keyframe-effects/transform-and-opacity-on-inline-001.html b/web-animations/animation-model/keyframe-effects/transform-and-opacity-on-inline-001.html
new file mode 100644
index 0000000..f76b53c
--- /dev/null
+++ b/web-animations/animation-model/keyframe-effects/transform-and-opacity-on-inline-001.html
@@ -0,0 +1,27 @@
+<!DOCTYPE html>
+<html>
+<title>CSS Test (Animations): Element.animate() animating both transform and opacity on an inline</title>
+<link rel="author" title="L. David Baron" href="https://dbaron.org/">
+<link rel="author" title="Google" href="http://www.google.com/">
+<link rel="help" href="https://bugs.chromium.org/p/chromium/issues/detail?id=1316688">
+<link rel="match" href="transform-and-opacity-on-inline-001-ref.html">
+<meta name="assert" content="This should not crash, and should render as opacity 0.5.">
+
+<script>
+// The transform animation should be ignored; the opacity animation should work.
+window.onload = function() {
+ document.getElementById("target").animate(
+ [
+ {
+ "transform": "translateX(0px)",
+ "opacity": "0.8",
+ },
+ {
+ "transform": "translateX(300px)",
+ "opacity": "0.0",
+ }
+ ],
+ { duration:1000000, delay: -500000, easing: "steps(3, jump-both)" });
+}
+</script>
+<body><span id="target">x</span></body>
