find_node_by_unique_id should use find

find_node_by_unique_id should use find instead of for-loop
servo · Feb 24, 2016 · cad3115 · cad3115
1 parent 83b2388
commit cad3115
Showing 1 changed file with 1 addition and 9 deletions.
diff --git 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<Page>, pipeline: PipelineId, node_id: String) -> Option<Root<Node>> {
     let page = get_page(&*page, pipeline);
     let document = page.document();
-    let node = document.upcast::<Node>();
-
-    for candidate in node.traverse_preorder() {
-        if candidate.get_unique_id() == node_id {
-            return Some(candidate);
-        }
-    }
-
-    None
+    document.upcast::<Node>().traverse_preorder().find(|candidate| candidate.get_unique_id() == node_id)
 }
 
 #[allow(unsafe_code)]