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

Pass the WorkQueue to LayoutTask::solve_constraints_parallel.

Browse source 
This removes the possibility of a panic by checking a constraint at compile
time rather than at run time.
This commit is contained in:
Ms2ger 2015-07-09 11:15:53 +02:00
parent 9e02ef9347
commit 17888045bd
1 changed files with 10 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

25
components/layout/layout_task.rs
View file

@ -757,23 +757,18 @@ impl LayoutTask {
/// benchmarked against those two. It is marked `#[inline(never)]` to aid profiling.
#[inline(never)]
fn solve_constraints_parallel(&self,
rw_data: &mut LayoutTaskData,
traversal: &mut WorkQueue<SharedLayoutContext, WorkQueueData>,
layout_root: &mut FlowRef,
shared_layout_context: &SharedLayoutContext) {
let _scope = layout_debug_scope!("solve_constraints_parallel");
match rw_data.parallel_traversal {
None => panic!("solve_contraints_parallel() called with no parallel traversal ready"),
Some(ref mut traversal) => {
// NOTE: this currently computes borders, so any pruning should separate that
// operation out.
parallel::traverse_flow_tree_preorder(layout_root,
self.profiler_metadata(),
self.time_profiler_chan.clone(),
shared_layout_context,
traversal);
}
}
// NOTE: this currently computes borders, so any pruning should separate that
// operation out.
parallel::traverse_flow_tree_preorder(layout_root,
self.profiler_metadata(),
self.time_profiler_chan.clone(),
shared_layout_context,
traversal);
}
/// Verifies that every node was either marked as a leaf or as a nonleaf in the flow tree.
@ -1152,9 +1147,9 @@ impl LayoutTask {
// Sequential mode.
self.solve_constraints(&mut root_flow, &layout_context)
}
Some(_) => {
Some(ref mut parallel) => {
// Parallel mode.
self.solve_constraints_parallel(rw_data,
self.solve_constraints_parallel(parallel,
&mut root_flow,
&mut *layout_context);
}