Disallow percentage values in number interpolations

lilles · chromium-wpt-export-bot · commit c87bc60fe53e · 2025-04-04T05:56:35.000-07:00
line-height and registered custom property "<number>|<percentage>" incorrectly mixed numbers and percentages in interpolations. The text-size-adjust accepts percentages, not numbers. It still worked using interpolable numbers, but it's more correct to make it a percentage interpolation type. Bug: 407800771, 408030109 Change-Id: I23fc95c9d208bf4b6f83f85164722b1df7df8724 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6425427 Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/main@{#1442670}
diff --git a/css/CSS2/linebox/animations/line-height-interpolation.html b/css/CSS2/linebox/animations/line-height-interpolation.html
@@ -183,4 +183,21 @@
   from: '14q',
   to: 'normal',
 });
+
+test_no_interpolation({
+  property: 'line-height',
+  from: '1',
+  to: '200%',
+});
+
+test_interpolation({
+  property: 'line-height',
+  from: '100%',
+  to: '200%',
+}, [
+  {at: 0, expect: '100%'},
+  {at: 0.3, expect: '130%'},
+  {at: 0.6, expect: '160%'},
+  {at: 1, expect: '200%'},
+]);
 </script>
diff --git a/css/css-properties-values-api/animation/custom-property-transition-number-percentage.html b/css/css-properties-values-api/animation/custom-property-transition-number-percentage.html
@@ -0,0 +1,16 @@
+<!DOCTYPE html>
+<title>CSS Properties and Values API Test: Transitions between percentages and numbers are discrete</title>
+<link rel="help" href="https://drafts.css-houdini.org/css-properties-values-api-1/">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="../resources/utils.js"></script>
+<div id="target"></div>
+<script>
+  transition_test({
+    syntax: "<number>|<percentage>",
+    from: "100",
+    to: "200%",
+    expected: "200%",
+    behavior: 'allow-discrete',
+  }, 'A custom property with <number>|<percentage> syntax should not transition between values of the different types');
+</script>
