Enable font-variant-ligatures in stylo

Signed-off-by: Simon Wülker <simon.wuelker@arcor.de>

Author: simonwuelker

style/properties/longhands.toml

@@ -991,7 +991,6 @@ affects = "layout"
 type = "FontVariantLigatures"
 initial = "computed::FontVariantLigatures::empty()"
 struct = "font"
-engine = "gecko"
 animation_type = "discrete"
 gecko_ffi_name = "mFont.variantLigatures"
 spec = "https://drafts.csswg.org/css-fonts/#propdef-font-variant-ligatures"

style/properties/shorthands.rs

@@ -2519,11 +2519,11 @@ pub mod font {
     use crate::properties::longhands::{
         font_family, font_feature_settings, font_language_override, font_size,
         font_size_adjust, font_variant_alternates, font_variant_east_asian, font_variant_emoji,
-        font_variant_ligatures, font_variant_numeric, font_variant_position,
+        font_variant_numeric, font_variant_position,
     };
     use crate::properties::longhands::{
-        font_kerning, font_optical_sizing, font_stretch, font_style, font_variant_caps, font_variation_settings,
-        font_weight,
+        font_kerning, font_variant_ligatures, font_optical_sizing, font_stretch, font_style, font_variant_caps,
+        font_variation_settings, font_weight,
     };
     #[cfg(feature = "gecko")]
     use crate::values::specified::font::SystemFont;
@@ -2634,7 +2634,6 @@ pub mod font {
             font_variant_east_asian: font_variant_east_asian::get_initial_specified_value(),
             #[cfg(feature = "gecko")]
             font_variant_emoji: font_variant_emoji::get_initial_specified_value(),
-            #[cfg(feature = "gecko")]
             font_variant_ligatures: font_variant_ligatures::get_initial_specified_value(),
             #[cfg(feature = "gecko")]
             font_variant_numeric: font_variant_numeric::get_initial_specified_value(),
@@ -2705,7 +2704,6 @@ pub mod font {
             {
                 return Ok(());
             }
-            #[cfg(feature = "gecko")]
             if self.font_variant_ligatures != &font_variant_ligatures::get_initial_specified_value()
             {
                 return Ok(());
@@ -2843,20 +2841,18 @@ pub mod font_variant {
     pub use crate::properties::generated::shorthands::font_variant::*;
 
     use super::*;
-    use crate::properties::longhands::font_variant_caps;
+    use crate::properties::longhands::{font_variant_caps, font_variant_ligatures};
     #[cfg(feature = "gecko")]
     use crate::properties::longhands::{
         font_variant_alternates, font_variant_east_asian, font_variant_emoji,
-        font_variant_ligatures, font_variant_numeric, font_variant_position,
+        font_variant_numeric, font_variant_position,
     };
-    #[allow(unused_imports)]
     use crate::values::specified::FontVariantLigatures;
 
     pub fn parse_value<'i, 't>(
         context: &ParserContext,
         input: &mut Parser<'i, 't>,
     ) -> Result<Longhands, ParseError<'i>> {
-        #[cfg(feature = "gecko")]
         let mut ligatures = None;
         let mut caps = None;
         #[cfg(feature = "gecko")]
@@ -2878,7 +2874,6 @@ pub mod font_variant {
             .try_parse(|input| input.expect_ident_matching("none"))
             .is_ok()
         {
-            #[cfg(feature = "gecko")]
             {
                 ligatures = Some(FontVariantLigatures::NONE);
             }
@@ -2895,7 +2890,6 @@ pub mod font_variant {
                 {
                     return Err(input.new_custom_error(StyleParseErrorKind::UnspecifiedError));
                 }
-                #[cfg(feature = "gecko")]
                 try_parse_one!(context, input, ligatures, font_variant_ligatures::parse);
                 try_parse_one!(context, input, caps, font_variant_caps::parse);
                 #[cfg(feature = "gecko")]
@@ -2929,6 +2923,7 @@ pub mod font_variant {
         });
         #[cfg(feature = "servo")]
         return Ok(expanded! {
+            font_variant_ligatures: unwrap_or_initial!(font_variant_ligatures, ligatures),
             font_variant_caps: unwrap_or_initial!(font_variant_caps, caps),
         });
     }
@@ -2939,15 +2934,12 @@ pub mod font_variant {
         where
             W: fmt::Write,
         {
-            #[cfg(feature = "gecko")]
             let has_none_ligatures = self.font_variant_ligatures == &FontVariantLigatures::NONE;
-            #[cfg(feature = "servo")]
-            let has_none_ligatures = false;
 
             #[cfg(feature = "gecko")]
             const TOTAL_SUBPROPS: usize = 7;
             #[cfg(feature = "servo")]
-            const TOTAL_SUBPROPS: usize = 1;
+            const TOTAL_SUBPROPS: usize = 2;
             let mut nb_normals = 0;
             macro_rules! count_normal {
                 ($e: expr, $p: ident) => {
@@ -2959,7 +2951,6 @@ pub mod font_variant {
                     count_normal!(self.$v, $v);
                 };
             }
-            #[cfg(feature = "gecko")]
             count_normal!(font_variant_ligatures);
             count_normal!(font_variant_caps);
             #[cfg(feature = "gecko")]
@@ -3001,7 +2992,6 @@ pub mod font_variant {
                 };
             }
 
-            #[cfg(feature = "gecko")]
             write!(font_variant_ligatures);
             write!(font_variant_caps);
             #[cfg(feature = "gecko")]

style/properties/shorthands.toml

@@ -430,23 +430,23 @@ spec = "https://drafts.csswg.org/css-text-decor/#propdef-text-decoration"
 
 [font]
 sub_properties = [
-  "font-style",
-  "font-variant-caps",
-  "font-weight",
-  "font-stretch",
-  "font-size",
-  "line-height",
   "font-family",
+  "font-kerning",
   "font-optical-sizing",
+  "font-size",
+  "font-stretch",
+  "font-style",
+  "font-variant-caps",
+  "font-variant-ligatures",
   "font-variation-settings",
-  "font-kerning"
+  "font-weight",
+  "line-height"
 ]
 extra_gecko_sub_properties = [
   "font-size-adjust",
   "font-variant-alternates",
   "font-variant-east-asian",
   "font-variant-emoji",
-  "font-variant-ligatures",
   "font-variant-numeric",
   "font-variant-position",
   "font-language-override",
@@ -456,8 +456,8 @@ spec = "https://drafts.csswg.org/css-fonts-3/#propdef-font"
 derive_value_info = false
 
 [font-variant]
-sub_properties = ["font-variant-caps"]
-extra_gecko_sub_properties = ["font-variant-alternates", "font-variant-east-asian", "font-variant-emoji", "font-variant-ligatures", "font-variant-numeric", "font-variant-position"]
+sub_properties = ["font-variant-caps", "font-variant-ligatures"]
+extra_gecko_sub_properties = ["font-variant-alternates", "font-variant-east-asian", "font-variant-emoji", "font-variant-numeric", "font-variant-position"]
 spec = "https://drafts.csswg.org/css-fonts-3/#propdef-font-variant"
 
 [font-synthesis]

style/values/specified/font.rs

@@ -1375,6 +1375,7 @@ impl FontVariantEastAsian {
     ToShmem,
     ToTyped,
 )]
+#[cfg_attr(feature = "servo", derive(Deserialize, Hash, Serialize))]
 #[css(bitflags(
     single = "normal,none",
     mixed = "common-ligatures,no-common-ligatures,discretionary-ligatures,no-discretionary-ligatures,historical-ligatures,no-historical-ligatures,contextual,no-contextual",