diff options
author | Corey Farwell <coreyf@rwell.org> | 2016-11-02 15:57:12 -0400 |
---|---|---|
committer | Corey Farwell <coreyf@rwell.org> | 2016-11-02 16:53:00 -0400 |
commit | 927d44753b249a2716b721d7398f4dc4a208b82f (patch) | |
tree | cba2307ecbb74845ed197254596e97286ae47faf /components/script/dom/node.rs | |
parent | 25e3ae6915bce0446a4bd1b1b89b2f0faaef2e02 (diff) | |
download | servo-927d44753b249a2716b721d7398f4dc4a208b82f.tar.gz servo-927d44753b249a2716b721d7398f4dc4a208b82f.zip |
Remove unnecessary `Box` around `HTMLCollectionElementsIter::node_iter`.
Diffstat (limited to 'components/script/dom/node.rs')
-rw-r--r-- | components/script/dom/node.rs | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index 767a3132eb3..8afe20f0aea 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -849,22 +849,23 @@ impl Node { let tr = new_child(); - let after_node = if index == -1 { - None - } else { - match get_items().elements_iter() - .map(Root::upcast::<Node>) - .map(Some) - .chain(iter::once(None)) - .nth(index as usize) { - None => return Err(Error::IndexSize), - Some(node) => node, - } - }; { let tr_node = tr.upcast::<Node>(); - try!(self.InsertBefore(tr_node, after_node.r())); + if index == -1 { + try!(self.InsertBefore(tr_node, None)); + } else { + let items = get_items(); + let node = match items.elements_iter() + .map(Root::upcast::<Node>) + .map(Some) + .chain(iter::once(None)) + .nth(index as usize) { + None => return Err(Error::IndexSize), + Some(node) => node, + }; + try!(self.InsertBefore(tr_node, node.r())); + } } Ok(Root::upcast::<HTMLElement>(tr)) |