Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Async scene building. #2362

Merged
merged 18 commits into from Feb 22, 2018
Merged
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Refactor transactions and forward them to the scene builder thread.

  • Loading branch information
nical committed Feb 21, 2018
commit 5df631e1b97cce9e01e062636f1b29ba62973108
@@ -1248,8 +1248,6 @@ impl FrameContext {
clip_scroll_tree,
pipeline_epoch_map,
removed_pipelines: replace(&mut scene.removed_pipelines, Vec::new()),
document_id: request.document_id,
render: request.render,
}
}

@@ -67,13 +67,21 @@ pub fn should_record_msg(msg: &ApiMsg) -> bool {
ApiMsg::AddDocument { .. } |
ApiMsg::DeleteDocument(..) => true,
ApiMsg::UpdateDocument(_, ref msgs) => {
for msg in msgs {
for msg in &msgs.prefix_ops {
match *msg {
DocumentMsg::GetScrollNodeState(..) |
DocumentMsg::HitTest(..) => {}
_ => { return true; }
}
}
for msg in &msgs.postfix_ops {
match *msg {
DocumentMsg::GetScrollNodeState(..) |
DocumentMsg::HitTest(..) => {}
_ => { return true; }
}
}

false
}
_ => false,
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.