tor-browser

commit 77300fb4d296994ab6c812c1facfc05dc30efaf8
parent 908469b74aca7221fae0e2634222de3c0188bad5
Author: Emily McDonough <emcdonough@mozilla.com>
Date:   Thu, 30 Oct 2025 19:08:06 +0000

Bug 1993043 - Enable layout.css.text-decoration-inset.enabled by default r=layout-jp-market-reviewers,jfkthame

This also fixes the entry in property_database.js, which has a few problems.
The first value of `other_values` is the same as the initial value, the value
`calc(0)` does not parse as a length and `calc(0px) 0` will just become `0`
as that is the shorted serialization of that value, and according to the
WebVTT spec text-decoration-inset should not apply to ::cue.

Differential Revision: https://phabricator.services.mozilla.com/D270377

Diffstat:
M
layout/style/test/property_database.js
 | 
4
+---
M
layout/style/test/test_transitions_per_property.html
 | 
6
++++++
M
modules/libpref/init/StaticPrefList.yaml
 | 
2
+-

3 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/layout/style/test/property_database.js b/layout/style/test/property_database.js
@@ -14449,10 +14449,8 @@ if (IsCSSPropertyPrefEnabled("layout.css.text-decoration-inset.enabled")) {
       applies_to_first_letter: true,
       applies_to_first_line: true,
       applies_to_placeholder: true,
-      applies_to_cue: true,
       initial_values: ["0"],
       other_values: [
-        "0",
         "-14px",
         "25px",
         "100em",
@@ -14465,7 +14463,7 @@ if (IsCSSPropertyPrefEnabled("layout.css.text-decoration-inset.enabled")) {
         "-1mm -2px",
         "calc(-1mm + 2em)",
         "2px calc(2px - 1em)",
-        "calc(0) 0",
+        "calc(1px) 0",
       ],
       invalid_values: [
         "13",
diff --git a/layout/style/test/test_transitions_per_property.html b/layout/style/test/test_transitions_per_property.html
@@ -376,6 +376,12 @@ if (IsCSSPropertyPrefEnabled("layout.css.zoom.enabled")) {
   });
 }
 
+if (IsCSSPropertyPrefEnabled("layout.css.text-decoration-inset.enabled")) {
+  Object.assign(supported_properties, {
+    "text-decoration-inset": [ test_length_transition ],
+  });
+}
+
 // For properties which are well-tested by web-platform-tests, we don't need to
 // test animations/transitions again on them.
 var skipped_transitionable_properties = [
diff --git a/modules/libpref/init/StaticPrefList.yaml b/modules/libpref/init/StaticPrefList.yaml
@@ -10449,7 +10449,7 @@
 
 - name: layout.css.text-decoration-inset.enabled
   type: RelaxedAtomicBool
-  value: false
+  value: true
   mirror: always
   rust: true