text-decoration shorthand should support style and color

[Loirooriol]

servo/servo@689b413 and servo/servo@928bd0d added support for text-decoration-style and text-decoration-color on Servo. However, the shorthand wasn't modified:

stylo/style/properties/shorthands/text.mako.rs

Lines 10 to 11 in 3d41c54

	sub_properties="text-decoration-line
	${' text-decoration-style text-decoration-color text-decoration-thickness' if engine == 'gecko' else ''}"

This seems something that we should fix after catching up with the style updates.

[MunishMummadi]

Hey @Loirooriol I tried to understand the code and I think sub_properties should be updated with.

sub_properties="text-decoration-line ${' text-decoration-style text-decoration-color' if engine == 'servo' else ''} ${' text-decoration-style text-decoration-color text-decoration-thickness' if engine == 'gecko' else ''}"

Let me know if I am wrong.

[mrobinson]

@MunishMummadi The proposal here was to wait until Stylo has been updated to incorporate all of the changes from upstream.

[Loirooriol]

Yeah. Well I guess it's not a big deal to fix it beforehand, but that code is not enough, the parser and serializer need to change too.

[MunishMummadi]

Got it!!!! thanks for info @Loirooriol @mrobinson