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

Properly parse alignment shorthands (fixes #16391)

Browse source
This commit is contained in:
Anthony Ramine 2017-04-20 12:54:36 +02:00
parent 7f825d2119
commit 88f682a5d5
2 changed files with 42 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

18
components/style/properties/shorthand/position.mako.rs
View file

@ -249,8 +249,14 @@
pub fn parse_value(context: &ParserContext, input: &mut Parser) -> Result<Longhands, ()> {
let align = align_content::parse(context, input)?;
if align.has_extra_flags() {
return Err(());
}
let justify = input.try(|input| justify_content::parse(context, input))
.unwrap_or(justify_content::SpecifiedValue::from(align));
if justify.has_extra_flags() {
return Err(());
}
Ok(Longhands {
align_content: align,
@ -279,7 +285,13 @@
pub fn parse_value(context: &ParserContext, input: &mut Parser) -> Result<Longhands, ()> {
let align = AlignJustifySelf::parse(context, input)?;
if align.has_extra_flags() {
return Err(());
}
let justify = input.try(|input| AlignJustifySelf::parse(context, input)).unwrap_or(align.clone());
if justify.has_extra_flags() {
return Err(());
}
Ok(Longhands {
align_self: align,
@ -314,8 +326,14 @@
pub fn parse_value(context: &ParserContext, input: &mut Parser) -> Result<Longhands, ()> {
let align = AlignItems::parse(context, input)?;
if align.has_extra_flags() {
return Err(());
}
let justify = input.try(|input| JustifyItems::parse(context, input))
.unwrap_or(JustifyItems::from(align));
if justify.has_extra_flags() {
return Err(());
}
Ok(Longhands {
align_items: align,

24
components/style/values/specified/align.rs
View file

@ -155,6 +155,12 @@ impl AlignJustifyContent {
AlignFlags::from_bits((self.0 >> ALIGN_ALL_SHIFT) as u8)
.expect("AlignJustifyContent must contain valid flags")
}
/// Whether this value has extra flags.
#[inline]
pub fn has_extra_flags(self) -> bool {
self.primary().intersects(ALIGN_FLAG_BITS) || self.fallback().intersects(ALIGN_FLAG_BITS)
}
}
impl ToCss for AlignJustifyContent {
@ -213,6 +219,12 @@ impl AlignJustifySelf {
pub fn auto() -> Self {
AlignJustifySelf(ALIGN_AUTO)
}
/// Whether this value has extra flags.
#[inline]
pub fn has_extra_flags(self) -> bool {
self.0.intersects(ALIGN_FLAG_BITS)
}
}
no_viewport_percentage!(AlignJustifySelf);
@ -251,6 +263,12 @@ impl AlignItems {
pub fn normal() -> Self {
AlignItems(ALIGN_NORMAL)
}
/// Whether this value has extra flags.
#[inline]
pub fn has_extra_flags(self) -> bool {
self.0.intersects(ALIGN_FLAG_BITS)
}
}
no_viewport_percentage!(AlignItems);
@ -289,6 +307,12 @@ impl JustifyItems {
pub fn auto() -> Self {
JustifyItems(ALIGN_AUTO)
}
/// Whether this value has extra flags.
#[inline]
pub fn has_extra_flags(self) -> bool {
self.0.intersects(ALIGN_FLAG_BITS)
}
}
no_viewport_percentage!(JustifyItems);