Bug 1390367 - Don't allow interpolating 'fill:none' with 'fill:none'. r?birtles
Bug 1390364 fixed this type interpolating, but the recently changes[1]
uses Animate derive for this type(SVGPaintKind), then Servo can interpolate
this types(i.e. fill:none and fill:none).
This patch add animation(error) hint to SVGPaintKind::None in order to
disallow this interpolation.
[1] https://github.com/servo/servo/pull/18239/files
MozReview-Commit-ID: Fpt3SYvHfMf
--- a/servo/components/style/values/generics/svg.rs
+++ b/servo/components/style/values/generics/svg.rs
@@ -30,16 +30,17 @@ pub struct SVGPaint<ColorType, UrlPaintS
/// Whereas the spec only allows PaintServer
/// to have a fallback, Gecko lets the context
/// properties have a fallback as well.
#[cfg_attr(feature = "servo", derive(HeapSizeOf))]
#[derive(Animate, Clone, ComputeSquaredDistance, Debug, PartialEq)]
#[derive(ToAnimatedValue, ToAnimatedZero, ToComputedValue, ToCss)]
pub enum SVGPaintKind<ColorType, UrlPaintServer> {
/// `none`
+ #[animation(error)]
None,
/// `<color>`
Color(ColorType),
/// `url(...)`
#[animation(error)]
PaintServer(UrlPaintServer),
/// `context-fill`
ContextFill,