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

Deduplicate the loop in iterate_through_fragments

Browse source
This commit is contained in:
Simon Sapin 2020-02-10 17:44:57 +01:00
parent ef8c51c4b1
commit d353e08529
1 changed files with 24 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files

22
components/layout_2020/flow/root.rs
View file

@ -204,14 +204,15 @@ impl FragmentTreeRoot {
where where
F: FnMut(&Fragment, &PhysicalRect<Length>) -> bool, F: FnMut(&Fragment, &PhysicalRect<Length>) -> bool,
{ {
fn do_iteration<M>( fn recur<M>(
fragment: &Fragment, fragments: &[Fragment],
containing_block: &PhysicalRect<Length>, containing_block: &PhysicalRect<Length>,
process_func: &mut M, process_func: &mut M,
) -> bool ) -> bool
where where
M: FnMut(&Fragment, &PhysicalRect<Length>) -> bool, M: FnMut(&Fragment, &PhysicalRect<Length>) -> bool,
{ {
for fragment in fragments {
if !process_func(fragment, containing_block) { if !process_func(fragment, containing_block) {
return false; return false;
} }
@ -222,34 +223,25 @@ impl FragmentTreeRoot {
.content_rect .content_rect
.to_physical(fragment.style.writing_mode, containing_block) .to_physical(fragment.style.writing_mode, containing_block)
.translate(containing_block.origin.to_vector()); .translate(containing_block.origin.to_vector());
for child in &fragment.children { if !recur(&fragment.children, &new_containing_block, process_func) {
if !do_iteration(child, &new_containing_block, process_func) {
return false; return false;
} }
}
}, },
Fragment::Anonymous(fragment) => { Fragment::Anonymous(fragment) => {
let new_containing_block = fragment let new_containing_block = fragment
.rect .rect
.to_physical(fragment.mode, containing_block) .to_physical(fragment.mode, containing_block)
.translate(containing_block.origin.to_vector()); .translate(containing_block.origin.to_vector());
for child in &fragment.children { if !recur(&fragment.children, &new_containing_block, process_func) {
if !do_iteration(child, &new_containing_block, process_func) {
return false; return false;
} }
}
}, },
_ => {}, _ => {},
} }
}
true true
} }
recur(&self.children, &self.initial_containing_block, process_func);
for child in &self.children {
if !do_iteration(child, &self.initial_containing_block, process_func) {
break;
}
}
} }
pub fn get_content_box_for_node(&self, requested_node: OpaqueNode) -> Rect<Au> { pub fn get_content_box_for_node(&self, requested_node: OpaqueNode) -> Rect<Au> {