Return the result from match_element rather than using an outparam.

This commit is contained in:
Ms2ger 2015-10-15 11:25:07 +02:00
parent 5c7dc4c633
commit 604601f1f4
2 changed files with 10 additions and 11 deletions

View file

@ -375,8 +375,8 @@ pub trait ElementMatchMethods {
fn match_element(&self,
stylist: &Stylist,
parent_bf: Option<&BloomFilter>,
applicable_declarations: &mut ApplicableDeclarations,
shareable: &mut bool);
applicable_declarations: &mut ApplicableDeclarations)
-> bool;
}
pub trait MatchMethods {
@ -549,8 +549,8 @@ impl<'ln> ElementMatchMethods for LayoutElement<'ln> {
fn match_element(&self,
stylist: &Stylist,
parent_bf: Option<&BloomFilter>,
applicable_declarations: &mut ApplicableDeclarations,
shareable: &mut bool) {
applicable_declarations: &mut ApplicableDeclarations)
-> bool {
let style_attribute = self.style_attribute().as_ref();
applicable_declarations.normal_shareable =
@ -570,9 +570,9 @@ impl<'ln> ElementMatchMethods for LayoutElement<'ln> {
Some(PseudoElement::After),
&mut applicable_declarations.after);
*shareable = applicable_declarations.normal_shareable &&
applicable_declarations.before.is_empty() &&
applicable_declarations.after.is_empty()
applicable_declarations.normal_shareable &&
applicable_declarations.before.is_empty() &&
applicable_declarations.after.is_empty()
}
}

View file

@ -177,10 +177,9 @@ impl<'a> PreorderDomTraversal for RecalcStyleForNode<'a> {
if let Some(element) = node.as_element() {
// Perform the CSS selector matching.
let stylist = unsafe { &*self.layout_context.shared.stylist };
element.match_element(stylist,
Some(&*bf),
&mut applicable_declarations,
&mut shareable);
shareable = element.match_element(stylist,
Some(&*bf),
&mut applicable_declarations);
} else if node.has_changed() {
ThreadSafeLayoutNode::new(&node).set_restyle_damage(
incremental::rebuild_and_reflow())