auto merge of #1138 : Ms2ger/servo/reflect-HTMLUnknownElement, r=jdm

@jdm: I think we should move nodes more in line with the rest of the DOM. How do you feel about something like this?
This commit is contained in:
bors-servo 2013-10-29 12:17:44 -07:00
commit 2c3d5ec79f
3 changed files with 33 additions and 5 deletions

View file

@ -5,7 +5,7 @@
//! The core DOM types. Defines the basic DOM hierarchy as well as all the HTML elements.
use dom::bindings::node;
use dom::bindings::utils::{Reflectable, Reflector};
use dom::bindings::utils::{Reflectable, Reflector, reflect_dom_object};
use dom::bindings::utils::{DOMString, null_str_as_empty};
use dom::bindings::utils::{ErrorResult, Fallible, NotFound, HierarchyRequest};
use dom::characterdata::CharacterData;
@ -485,6 +485,20 @@ impl Node<ScriptView> {
node
}
pub fn reflect_node<N: Reflectable>
(node: @mut N,
document: AbstractDocument,
wrap_fn: extern "Rust" fn(*JSContext, *JSObject, @mut N) -> *JSObject)
-> AbstractNode<ScriptView> {
assert!(node.reflector().get_jsobject().is_null());
let node = reflect_dom_object(node, document.document().window, wrap_fn);
assert!(node.reflector().get_jsobject().is_not_null());
// This surrenders memory management of the node!
AbstractNode {
obj: unsafe { transmute(node) },
}
}
pub fn add_to_doc(&mut self, abstract_self: AbstractNode<ScriptView>, doc: AbstractDocument) {
let old_doc = self.owner_doc();
self.set_owner_doc(doc);