style: Remove support for -moz-system-color() as chrome code no longer uses it (in favor of color-scheme)

Differential Revision: https://phabricator.services.mozilla.com/D127802
This commit is contained in:
Emilio Cobos Álvarez 2023-05-27 16:13:46 +02:00 committed by Oriol Brufau
parent 5766329ffe
commit b3deee4cc8

View file

@ -194,20 +194,6 @@ impl ToCss for ColorMix {
} }
} }
/// The color scheme for a specific system color.
#[cfg(feature = "gecko")]
#[derive(Clone, Copy, Debug, MallocSizeOf, Parse, PartialEq, ToCss, ToShmem)]
#[repr(u8)]
pub enum SystemColorScheme {
/// The default color-scheme for the document.
#[css(skip)]
Default,
/// A light color scheme.
Light,
/// A dark color scheme.
Dark,
}
/// Specified color value /// Specified color value
#[derive(Clone, Debug, MallocSizeOf, PartialEq, ToShmem)] #[derive(Clone, Debug, MallocSizeOf, PartialEq, ToShmem)]
pub enum Color { pub enum Color {
@ -222,10 +208,9 @@ pub enum Color {
}, },
/// A complex color value from computed value /// A complex color value from computed value
Complex(ComputedColor), Complex(ComputedColor),
/// Either a system color, or a `-moz-system-color(<system-color>, light|dark)` /// A system color.
/// function which allows chrome code to choose between color schemes.
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
System(SystemColor, SystemColorScheme), System(SystemColor),
/// A color mix. /// A color mix.
ColorMix(Box<ColorMix>), ColorMix(Box<ColorMix>),
/// Quirksmode-only rule for inheriting color from the body /// Quirksmode-only rule for inheriting color from the body
@ -480,7 +465,7 @@ pub enum SystemColor {
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
impl SystemColor { impl SystemColor {
#[inline] #[inline]
fn compute(&self, cx: &Context, scheme: SystemColorScheme) -> ComputedColor { fn compute(&self, cx: &Context) -> ComputedColor {
use crate::gecko_bindings::bindings; use crate::gecko_bindings::bindings;
let colors = &cx.device().pref_sheet_prefs().mColors; let colors = &cx.device().pref_sheet_prefs().mColors;
@ -497,7 +482,7 @@ impl SystemColor {
_ => { _ => {
let color = unsafe { let color = unsafe {
bindings::Gecko_GetLookAndFeelSystemColor(*self as i32, cx.device().document(), scheme, &style_color_scheme) bindings::Gecko_GetLookAndFeelSystemColor(*self as i32, cx.device().document(), &style_color_scheme)
}; };
if color == bindings::NS_SAME_AS_FOREGROUND_COLOR { if color == bindings::NS_SAME_AS_FOREGROUND_COLOR {
return ComputedColor::currentcolor(); return ComputedColor::currentcolor();
@ -578,21 +563,6 @@ impl<'a, 'b: 'a, 'i: 'a> ::cssparser::ColorComponentParser<'i> for ColorComponen
} }
} }
#[cfg(feature = "gecko")]
fn parse_moz_system_color<'i, 't>(
context: &ParserContext,
input: &mut Parser<'i, 't>,
) -> Result<(SystemColor, SystemColorScheme), ParseError<'i>> {
debug_assert!(context.chrome_rules_enabled());
input.expect_function_matching("-moz-system-color")?;
input.parse_nested_block(|input| {
let color = SystemColor::parse(context, input)?;
input.expect_comma()?;
let scheme = SystemColorScheme::parse(input)?;
Ok((color, scheme))
})
}
impl Parse for Color { impl Parse for Color {
fn parse<'i, 't>( fn parse<'i, 't>(
context: &ParserContext, context: &ParserContext,
@ -618,15 +588,7 @@ impl Parse for Color {
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
{ {
if let Ok(system) = input.try_parse(|i| SystemColor::parse(context, i)) { if let Ok(system) = input.try_parse(|i| SystemColor::parse(context, i)) {
return Ok(Color::System(system, SystemColorScheme::Default)); return Ok(Color::System(system));
}
if context.chrome_rules_enabled() {
if let Ok((color, scheme)) =
input.try_parse(|i| parse_moz_system_color(context, i))
{
return Ok(Color::System(color, scheme));
}
} }
} }
@ -665,17 +627,7 @@ impl ToCss for Color {
Color::Complex(_) => Ok(()), Color::Complex(_) => Ok(()),
Color::ColorMix(ref mix) => mix.to_css(dest), Color::ColorMix(ref mix) => mix.to_css(dest),
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
Color::System(system, scheme) => { Color::System(system) => system.to_css(dest),
if scheme == SystemColorScheme::Default {
system.to_css(dest)
} else {
dest.write_str("-moz-system-color(")?;
system.to_css(dest)?;
dest.write_str(", ")?;
scheme.to_css(dest)?;
dest.write_char(')')
}
},
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
Color::InheritFromBodyQuirk => Ok(()), Color::InheritFromBodyQuirk => Ok(()),
} }
@ -843,7 +795,7 @@ impl Color {
)) ))
}, },
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
Color::System(system, scheme) => system.compute(context?, scheme), Color::System(system) => system.compute(context?),
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
Color::InheritFromBodyQuirk => ComputedColor::rgba(context?.device().body_text_color()), Color::InheritFromBodyQuirk => ComputedColor::rgba(context?.device().body_text_color()),
}) })