Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-06 06:00:15 +01:00
Code Issues Projects Releases Packages Wiki Activity

Fix single_value_to_css

Browse source
This commit is contained in:
Manish Goregaokar 2016-11-13 22:39:38 -08:00
parent 1fff47586f
commit c387eff065
2 changed files with 9 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

17
components/style/properties/declaration_block.rs
View file

@ -214,16 +214,13 @@ impl PropertyDeclarationBlock {
if !self.declarations.iter().all(|decl| decl.0.shorthands().contains(&shorthand)) {
return Err(fmt::Error)
}
let success = try!(shorthand.serialize_shorthand_to_buffer(
dest,
self.declarations.iter()
.map(get_declaration as fn(_) -> _),
&mut true)
);
if success {
Ok(())
} else {
Err(fmt::Error)
let iter = self.declarations.iter().map(get_declaration as fn(_) -> _);
match shorthand.get_shorthand_appendable_value(iter) {
Some(AppendableValue::Css(css)) => dest.write_str(css),
Some(AppendableValue::DeclarationsForShorthand(_, decls)) => {
shorthand.longhands_to_css(decls, dest)
}
_ => Ok(())
}
}
}

4
tests/unit/style/properties/serialization.rs
View file

@ -1030,7 +1030,7 @@ mod shorthand_serialization {
let x = block.single_value_to_css("scroll-snap-type", &mut s);
assert_eq!(x.is_ok(), true);
assert_eq!(s, "scroll-snap-type: ;");
assert_eq!(s, "");
}
#[test]
@ -1052,7 +1052,7 @@ mod shorthand_serialization {
let x = block.single_value_to_css("scroll-snap-type", &mut s);
assert_eq!(x.is_ok(), true);
assert_eq!(s, "scroll-snap-type: mandatory;");
assert_eq!(s, "mandatory");
}
}
}