Auto merge of #8935 - Ms2ger:get_or_create, r=Manishearth

Avoid some rooting in parsing.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8935)
<!-- Reviewable:end -->

2 files changed, 2 insertions, 34 deletions

diff --git a/components/script/dom/servohtmlparser.rs b/components/script/dom/servohtmlparser.rs
index 6584dbe5cb5..82ed85a9a9c 100644
--- a/components/script/dom/servohtmlparser.rs
+++ b/components/script/dom/servohtmlparser.rs
@@ -16,13 +16,12 @@ use dom::bindings::trace::JSTraceable;
 use dom::document::Document;
 use dom::node::Node;
 use dom::servoxmlparser::ServoXMLParser;
-use dom::text::Text;
 use dom::window::Window;
 use encoding::all::UTF_8;
 use encoding::types::{DecoderTrap, Encoding};
 use html5ever::tokenizer;
 use html5ever::tree_builder;
-use html5ever::tree_builder::{NodeOrText, TreeBuilder, TreeBuilderOpts};
+use html5ever::tree_builder::{TreeBuilder, TreeBuilderOpts};
 use hyper::header::ContentType;
 use hyper::mime::{Mime, SubLevel, TopLevel};
 use js::jsapi::JSTracer;
@@ -36,7 +35,6 @@ use std::cell::UnsafeCell;
 use std::default::Default;
 use std::ptr;
 use url::Url;
-use util::str::DOMString;
 
 #[must_root]
 #[derive(JSTraceable, HeapSizeOf)]
@@ -45,21 +43,6 @@ pub struct Sink {
     pub document: JS<Document>,
 }
 
-impl Sink {
-    #[allow(unrooted_must_root)] // method is only run at parse time
-    pub fn get_or_create(&self, child: NodeOrText<JS<Node>>) -> Root<Node> {
-        match child {
-            NodeOrText::AppendNode(n) => Root::from_ref(&*n),
-            NodeOrText::AppendText(t) => {
-                // FIXME(ajeffrey): convert directly from tendrils to DOMStrings
-                let s: String = t.into();
-                let text = Text::new(DOMString::from(s), &self.document);
-                Root::upcast(text)
-            }
-        }
-    }
-}
-
 /// FragmentContext is used only to pass this group of related values
 /// into functions.
 #[derive(Copy, Clone)]
diff --git a/components/script/dom/servoxmlparser.rs b/components/script/dom/servoxmlparser.rs
index 4f4b4be7d3e..1433e9f49e1 100644
--- a/components/script/dom/servoxmlparser.rs
+++ b/components/script/dom/servoxmlparser.rs
@@ -11,7 +11,6 @@ use dom::bindings::trace::JSTraceable;
 use dom::document::Document;
 use dom::node::Node;
 use dom::servohtmlparser::ParserRef;
-use dom::text::Text;
 use dom::window::Window;
 use js::jsapi::JSTracer;
 use msg::constellation_msg::PipelineId;
@@ -19,9 +18,8 @@ use parse::Parser;
 use script_task::ScriptTask;
 use std::cell::Cell;
 use url::Url;
-use util::str::DOMString;
 use xml5ever::tokenizer;
-use xml5ever::tree_builder::{self, NodeOrText, XmlTreeBuilder};
+use xml5ever::tree_builder::{self, XmlTreeBuilder};
 
 pub type Tokenizer = tokenizer::XmlTokenizer<XmlTreeBuilder<JS<Node>, Sink>>;
 
@@ -32,19 +30,6 @@ pub struct Sink {
     pub document: JS<Document>,
 }
 
-impl Sink {
-    #[allow(unrooted_must_root)] // method is only run at parse time
-    pub fn get_or_create(&self, child: NodeOrText<JS<Node>>) -> Root<Node> {
-        match child {
-            NodeOrText::AppendNode(n) => Root::from_ref(&*n),
-            NodeOrText::AppendText(t) => {
-                let s: String = t.into();
-                let text = Text::new(DOMString::from(s), &self.document);
-                Root::upcast(text)
-            }
-        }
-    }
-}
 #[must_root]
 #[dom_struct]
 pub struct ServoXMLParser {