Auto merge of #16938 - emilio:traversal-assert, r=bzbarsky

stylo: Don't mess up traversal drivers.

Right now it's harmless, but it trips assertions that are nice to have, just in
case it stops being the case.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/16938)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2017-05-18 13:38:41 -05:00 committed by GitHub
commit e1ecd3b89c

View file

@ -210,14 +210,14 @@ fn traverse_subtree(element: GeckoElement,
debug!("Traversing subtree:");
debug!("{:?}", ShowSubtreeData(element.as_node()));
let traversal_driver = if global_style_data.style_thread_pool.is_none() {
let traversal_driver = if global_style_data.style_thread_pool.is_none() || !element.is_root() {
TraversalDriver::Sequential
} else {
TraversalDriver::Parallel
};
let traversal = RecalcStyleOnly::new(shared_style_context, traversal_driver);
if traversal_driver.is_parallel() && element.is_root() {
if traversal_driver.is_parallel() {
parallel::traverse_dom(&traversal, element, token,
global_style_data.style_thread_pool.as_ref().unwrap());
} else {