diff options
author | Anthony Ramine <n.oxyde@gmail.com> | 2015-10-07 14:55:02 +0200 |
---|---|---|
committer | Anthony Ramine <n.oxyde@gmail.com> | 2015-10-21 11:40:34 +0200 |
commit | 68014af78e8e3f5de4df0f6cc4d63b99c77478f5 (patch) | |
tree | f65b1a66ad8d7ce65042e37cf654da75e1766939 /components/script/dom/domimplementation.rs | |
parent | 13ea3ac413c511872784ccde416956217746553c (diff) | |
download | servo-68014af78e8e3f5de4df0f6cc4d63b99c77478f5.tar.gz servo-68014af78e8e3f5de4df0f6cc4d63b99c77478f5.zip |
Clean up the cast calls
Diffstat (limited to 'components/script/dom/domimplementation.rs')
-rw-r--r-- | components/script/dom/domimplementation.rs | 27 |
1 files changed, 10 insertions, 17 deletions
diff --git a/components/script/dom/domimplementation.rs b/components/script/dom/domimplementation.rs index af94a32b854..02c8ad4fa84 100644 --- a/components/script/dom/domimplementation.rs +++ b/components/script/dom/domimplementation.rs @@ -80,20 +80,13 @@ impl DOMImplementationMethods for DOMImplementation { let doc_node = doc.upcast::<Node>(); // Step 4. - match maybe_doctype { - None => (), - Some(ref doctype) => { - let doc_type = doctype.upcast::<Node>(); - assert!(doc_node.AppendChild(doc_type).is_ok()) - } + if let Some(doc_type) = maybe_doctype { + doc_node.AppendChild(doc_type.upcast()).unwrap(); } // Step 5. - match maybe_elem { - None => (), - Some(ref elem) => { - assert!(doc_node.AppendChild(elem.upcast::<Node>()).is_ok()) - } + if let Some(ref elem) = maybe_elem { + doc_node.AppendChild(elem.upcast()).unwrap(); } } @@ -117,7 +110,7 @@ impl DOMImplementationMethods for DOMImplementation { // Step 3. let doc_node = doc.upcast::<Node>(); let doc_type = DocumentType::new("html".to_owned(), None, None, doc.r()); - assert!(doc_node.AppendChild(doc_type.upcast::<Node>()).is_ok()); + doc_node.AppendChild(doc_type.upcast()).unwrap(); } { @@ -125,13 +118,13 @@ impl DOMImplementationMethods for DOMImplementation { let doc_node = doc.upcast::<Node>(); let doc_html = Root::upcast::<Node>( HTMLHtmlElement::new("html".to_owned(), None, doc.r())); - assert!(doc_node.AppendChild(doc_html.r()).is_ok()); + doc_node.AppendChild(&doc_html).expect("Appending failed"); { // Step 5. let doc_head = Root::upcast::<Node>( HTMLHeadElement::new("head".to_owned(), None, doc.r())); - assert!(doc_html.r().AppendChild(doc_head.r()).is_ok()); + doc_html.AppendChild(&doc_head).unwrap(); // Step 6. match title { @@ -140,18 +133,18 @@ impl DOMImplementationMethods for DOMImplementation { // Step 6.1. let doc_title = Root::upcast::<Node>( HTMLTitleElement::new("title".to_owned(), None, doc.r())); - assert!(doc_head.r().AppendChild(doc_title.r()).is_ok()); + doc_head.AppendChild(&doc_title).unwrap(); // Step 6.2. let title_text = Text::new(title_str, doc.r()); - assert!(doc_title.r().AppendChild(title_text.upcast::<Node>()).is_ok()); + doc_title.AppendChild(title_text.upcast()).unwrap(); } } } // Step 7. let doc_body = HTMLBodyElement::new("body".to_owned(), None, doc.r()); - assert!(doc_html.r().AppendChild(doc_body.upcast::<Node>()).is_ok()); + doc_html.AppendChild(doc_body.upcast()).unwrap(); } // Step 8. |