make -webkit-text-stroke color and width shorthand no longer sensitive to order of values

components/style/properties/shorthand/inherited_text.mako.rs

@@ -76,25 +76,34 @@
         use values::specified::{BorderWidth, Length};
         use app_units::Au;
 
-        let (mut color, mut width, mut any) = (None, None, false);
+        let mut color = None;
-        % for value in "color width".split():
+        let mut width = None;
-            if ${value}.is_none() {
+        loop {
-                if let Ok(value) = input.try(|input| _webkit_text_stroke_${value}::parse(context, input)) {
+            if color.is_none() {
-                    ${value} = Some(value);
+                if let Ok(value) = input.try(|input| _webkit_text_stroke_color::parse(context, input)) {
-                    any = true;
+                    color = Some(value);
+                    continue
                 }
             }
-        % endfor
 
-        if !any {
+            if width.is_none() {
-            return Err(());
+                if let Ok(value) = input.try(|input| _webkit_text_stroke_width::parse(context, input)) {
+                    width = Some(value);
+                    continue
+                }
+            }
+            break
         }
 
-        Ok(Longhands {
+        if color.is_some() || width.is_some() {
-            _webkit_text_stroke_color: color.or(Some(CSSColor { parsed: CSSParserColor::CurrentColor,
+            Ok(Longhands {
-                                                                authored: None })),
+                _webkit_text_stroke_color: color.or(Some(CSSColor { parsed: CSSParserColor::CurrentColor,
-            _webkit_text_stroke_width: width.or(Some(BorderWidth::from_length(Length::Absolute(Au::from_px(0))))),
+                    authored: None })),
-        })
+                _webkit_text_stroke_width: width.or(Some(BorderWidth::from_length(Length::Absolute(Au::from_px(0))))),
+            })
+        } else {
+            Err(())
+        }
     }
 
     impl<'a> LonghandsToSerialize<'a>  {

tests/unit/style/parsing/inherited_text.rs

@@ -78,3 +78,26 @@ fn test_text_emphasis_position() {
     let left_under = parse_longhand!(text_emphasis_position, "left under");
     assert_eq!(left_under, SpecifiedValue(HorizontalWritingModeValue::Under, VerticalWritingModeValue::Left));
 }
+
+
+#[test]
+fn webkit_text_stroke_shorthand_should_parse_properly() {
+    use style::properties::longhands::_webkit_text_stroke_color;
+    use style::properties::longhands::_webkit_text_stroke_width;
+    use style::properties::shorthands::_webkit_text_stroke;
+    use media_queries::CSSErrorReporterTest;
+    use servo_url::ServoUrl;
+    let url = ServoUrl::parse("http://localhost").unwrap();
+    let context = ParserContext::new(Origin::Author, &url, Box::new(CSSErrorReporterTest));
+
+    let mut parser = Parser::new("thin red");
+    let result = _webkit_text_stroke::parse_value(&context, &mut parser).unwrap();
+    assert_eq!(result._webkit_text_stroke_color.unwrap(), parse_longhand!(_webkit_text_stroke_color, "red"));
+    assert_eq!(result._webkit_text_stroke_width.unwrap(), parse_longhand!(_webkit_text_stroke_width, "thin"));
+
+    // ensure its no longer sensitive to order
+    let mut parser = Parser::new("red thin");
+    let result = _webkit_text_stroke::parse_value(&context, &mut parser).unwrap();
+    assert_eq!(result._webkit_text_stroke_color.unwrap(), parse_longhand!(_webkit_text_stroke_color, "red"));
+    assert_eq!(result._webkit_text_stroke_width.unwrap(), parse_longhand!(_webkit_text_stroke_width, "thin"));
+}