From cad31157085fe1d2b8a222fb369625867b963179 Mon Sep 17 00:00:00 2001 From: Daniel Robertson Date: Wed, 24 Feb 2016 13:55:52 +0000 Subject: [PATCH] find_node_by_unique_id should use find find_node_by_unique_id should use find instead of for-loop --- components/script/webdriver_handlers.rs | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/components/script/webdriver_handlers.rs b/components/script/webdriver_handlers.rs index 1b1d92fc914..e51a2e23804 100644 --- a/components/script/webdriver_handlers.rs +++ b/components/script/webdriver_handlers.rs @@ -40,15 +40,7 @@ use util::str::DOMString; fn find_node_by_unique_id(page: &Rc, pipeline: PipelineId, node_id: String) -> Option> { let page = get_page(&*page, pipeline); let document = page.document(); - let node = document.upcast::(); - - for candidate in node.traverse_preorder() { - if candidate.get_unique_id() == node_id { - return Some(candidate); - } - } - - None + document.upcast::().traverse_preorder().find(|candidate| candidate.get_unique_id() == node_id) } #[allow(unsafe_code)]