diff options
Diffstat (limited to 'components')
61 files changed, 479 insertions, 471 deletions
diff --git a/components/script/devtools.rs b/components/script/devtools.rs index c92098db2be..c2f0957f9ec 100644 --- a/components/script/devtools.rs +++ b/components/script/devtools.rs @@ -25,8 +25,8 @@ pub fn handle_evaluate_js(page: &Rc<Page>, pipeline: PipelineId, eval: String, r let page = get_page(&*page, pipeline); let frame = page.frame(); let window = frame.as_ref().unwrap().window.root(); - let cx = window.get_cx(); - let rval = window.evaluate_js_with_result(eval.as_slice()); + let cx = window.r().get_cx(); + let rval = window.r().evaluate_js_with_result(eval.as_slice()); reply.send(if rval.is_undefined() { devtools_traits::VoidValue @@ -51,7 +51,7 @@ pub fn handle_get_root_node(page: &Rc<Page>, pipeline: PipelineId, reply: Sender let frame = page.frame(); let document = frame.as_ref().unwrap().document.root(); - let node: JSRef<Node> = NodeCast::from_ref(*document); + let node: JSRef<Node> = NodeCast::from_ref(document.r()); reply.send(node.summarize()); } @@ -59,9 +59,9 @@ pub fn handle_get_document_element(page: &Rc<Page>, pipeline: PipelineId, reply: let page = get_page(&*page, pipeline); let frame = page.frame(); let document = frame.as_ref().unwrap().document.root(); - let document_element = document.GetDocumentElement().root().unwrap(); + let document_element = document.r().GetDocumentElement().root().unwrap(); - let node: JSRef<Node> = NodeCast::from_ref(*document_element); + let node: JSRef<Node> = NodeCast::from_ref(document_element.r()); reply.send(node.summarize()); } @@ -69,7 +69,7 @@ fn find_node_by_unique_id(page: &Rc<Page>, pipeline: PipelineId, node_id: String let page = get_page(&*page, pipeline); let frame = page.frame(); let document = frame.as_ref().unwrap().document.root(); - let node: JSRef<Node> = NodeCast::from_ref(*document); + let node: JSRef<Node> = NodeCast::from_ref(document.r()); for candidate in node.traverse_preorder() { if candidate.get_unique_id().as_slice() == node_id.as_slice() { @@ -82,20 +82,20 @@ fn find_node_by_unique_id(page: &Rc<Page>, pipeline: PipelineId, node_id: String pub fn handle_get_children(page: &Rc<Page>, pipeline: PipelineId, node_id: String, reply: Sender<Vec<NodeInfo>>) { let parent = find_node_by_unique_id(&*page, pipeline, node_id).root(); - let children = parent.children().map(|child| child.summarize()).collect(); + let children = parent.r().children().map(|child| child.summarize()).collect(); reply.send(children); } pub fn handle_get_layout(page: &Rc<Page>, pipeline: PipelineId, node_id: String, reply: Sender<(f32, f32)>) { let node = find_node_by_unique_id(&*page, pipeline, node_id).root(); - let elem: JSRef<Element> = ElementCast::to_ref(*node).expect("should be getting layout of element"); + let elem: JSRef<Element> = ElementCast::to_ref(node.r()).expect("should be getting layout of element"); let rect = elem.GetBoundingClientRect().root(); - reply.send((rect.Width(), rect.Height())); + reply.send((rect.r().Width(), rect.r().Height())); } pub fn handle_modify_attribute(page: &Rc<Page>, pipeline: PipelineId, node_id: String, modifications: Vec<Modification>) { let node = find_node_by_unique_id(&*page, pipeline, node_id).root(); - let elem: JSRef<Element> = ElementCast::to_ref(*node).expect("should be getting layout of element"); + let elem: JSRef<Element> = ElementCast::to_ref(node.r()).expect("should be getting layout of element"); for modification in modifications.iter(){ match modification.newValue { diff --git a/components/script/dom/activation.rs b/components/script/dom/activation.rs index 628cbe76016..6299abaa672 100644 --- a/components/script/dom/activation.rs +++ b/components/script/dom/activation.rs @@ -32,22 +32,23 @@ pub trait Activatable : Copy { fn synthetic_click_activation(&self, ctrlKey: bool, shiftKey: bool, altKey: bool, metaKey: bool) { let element = self.as_element().root(); // Step 1 - if element.click_in_progress() { + if element.r().click_in_progress() { return; } // Step 2 - element.set_click_in_progress(true); + element.r().set_click_in_progress(true); // Step 3 self.pre_click_activation(); // Step 4 // https://html.spec.whatwg.org/multipage/webappapis.html#fire-a-synthetic-mouse-event - let win = window_from_node(*element).root(); - let target: JSRef<EventTarget> = EventTargetCast::from_ref(*element); - let mouse = MouseEvent::new(*win, "click".into_string(), false, false, Some(*win), 1, + let win = window_from_node(element.r()).root(); + let target: JSRef<EventTarget> = EventTargetCast::from_ref(element.r()); + let mouse = MouseEvent::new(win.r(), "click".into_string(), + false, false, Some(win.r()), 1, 0, 0, 0, 0, ctrlKey, shiftKey, altKey, metaKey, 0, None).root(); - let event: JSRef<Event> = EventCast::from_ref(*mouse); + let event: JSRef<Event> = EventCast::from_ref(mouse.r()); event.set_trusted(true); target.dispatch_event(event); @@ -60,6 +61,6 @@ pub trait Activatable : Copy { } // Step 6 - element.set_click_in_progress(false); + element.r().set_click_in_progress(false); } } diff --git a/components/script/dom/attr.rs b/components/script/dom/attr.rs index 34f1d48c5a9..028a91ea422 100644 --- a/components/script/dom/attr.rs +++ b/components/script/dom/attr.rs @@ -151,8 +151,8 @@ impl<'a> AttrMethods for JSRef<'a, Attr> { } Some(o) => { let owner = o.root(); - let value = owner.parse_attribute(&self.namespace, self.local_name(), value); - self.set_value(AttrSettingType::ReplacedAttr, value, *owner); + let value = owner.r().parse_attribute(&self.namespace, self.local_name(), value); + self.set_value(AttrSettingType::ReplacedAttr, value, owner.r()); } } } diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index 793a4b1ef24..127e8615b1d 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -2199,7 +2199,7 @@ class CGCallGenerator(CGThing): if static: call = CGWrapper(call, pre="%s::" % descriptorProvider.interface.identifier.name) else: - call = CGWrapper(call, pre="%s." % object) + call = CGWrapper(call, pre="%s.r()." % object) call = CGList([call, CGWrapper(args, pre="(", post=")")]) self.cgRoot.append(CGList([ @@ -2214,7 +2214,7 @@ class CGCallGenerator(CGThing): if static: glob = "" else: - glob = " let global = global_object_for_js_object(this.reflector().get_jsobject());\n"\ + glob = " let global = global_object_for_js_object(this.r().reflector().get_jsobject());\n"\ " let global = global.root();\n" self.cgRoot.append(CGGeneric( diff --git a/components/script/dom/bindings/conversions.rs b/components/script/dom/bindings/conversions.rs index 068f4287206..b6ea44a6521 100644 --- a/components/script/dom/bindings/conversions.rs +++ b/components/script/dom/bindings/conversions.rs @@ -473,7 +473,7 @@ impl<T: Reflectable+IDLInterface> FromJSValConvertible<()> for JS<T> { impl<T: Reflectable> ToJSValConvertible for Root<T> { fn to_jsval(&self, cx: *mut JSContext) -> JSVal { - self.reflector().to_jsval(cx) + self.r().reflector().to_jsval(cx) } } diff --git a/components/script/dom/bindings/utils.rs b/components/script/dom/bindings/utils.rs index 9cf2c3eea47..7f5fa56a771 100644 --- a/components/script/dom/bindings/utils.rs +++ b/components/script/dom/bindings/utils.rs @@ -552,7 +552,7 @@ pub extern fn outerize_global(_cx: *mut JSContext, obj: JSHandleObject) -> *mut debug!("outerizing"); let obj = *obj.unnamed_field1; let win: Root<window::Window> = unwrap_jsmanaged(obj).unwrap().root(); - win.browser_context().as_ref().unwrap().window_proxy() + win.r().browser_context().as_ref().unwrap().window_proxy() } } diff --git a/components/script/dom/browsercontext.rs b/components/script/dom/browsercontext.rs index 0bd96b75ec8..7ac5f233246 100644 --- a/components/script/dom/browsercontext.rs +++ b/components/script/dom/browsercontext.rs @@ -39,7 +39,7 @@ impl BrowserContext { pub fn active_window(&self) -> Temporary<Window> { let doc = self.active_document().root(); - doc.window() + doc.r().window() } pub fn window_proxy(&self) -> *mut JSObject { @@ -55,7 +55,7 @@ impl BrowserContext { let WindowProxyHandler(handler) = js_info.as_ref().unwrap().dom_static.windowproxy_handler; assert!(handler.is_not_null()); - let parent = win.reflector().get_jsobject(); + let parent = win.r().reflector().get_jsobject(); let cx = js_info.as_ref().unwrap().js_context.ptr; let wrapper = with_compartment(cx, parent, || unsafe { WrapperNew(cx, parent, handler) diff --git a/components/script/dom/comment.rs b/components/script/dom/comment.rs index 82150bebf4e..40d9a5da9b6 100644 --- a/components/script/dom/comment.rs +++ b/components/script/dom/comment.rs @@ -40,7 +40,7 @@ impl Comment { pub fn Constructor(global: &GlobalRef, data: DOMString) -> Fallible<Temporary<Comment>> { let document = global.as_window().Document().root(); - Ok(Comment::new(data, *document)) + Ok(Comment::new(data, document.r())) } #[inline] diff --git a/components/script/dom/cssstyledeclaration.rs b/components/script/dom/cssstyledeclaration.rs index 2e20b85f942..f6e4fd861ed 100644 --- a/components/script/dom/cssstyledeclaration.rs +++ b/components/script/dom/cssstyledeclaration.rs @@ -85,13 +85,13 @@ trait PrivateCSSStyleDeclarationHelpers { impl<'a> PrivateCSSStyleDeclarationHelpers for JSRef<'a, CSSStyleDeclaration> { fn get_declaration(self, property: &Atom) -> Option<PropertyDeclaration> { let owner = self.owner.root(); - let element: JSRef<Element> = ElementCast::from_ref(*owner); + let element: JSRef<Element> = ElementCast::from_ref(owner.r()); element.get_inline_style_declaration(property).map(|decl| decl.clone()) } fn get_important_declaration(self, property: &Atom) -> Option<PropertyDeclaration> { let owner = self.owner.root(); - let element: JSRef<Element> = ElementCast::from_ref(*owner); + let element: JSRef<Element> = ElementCast::from_ref(owner.r()); element.get_important_inline_style_declaration(property).map(|decl| decl.clone()) } } @@ -99,7 +99,7 @@ impl<'a> PrivateCSSStyleDeclarationHelpers for JSRef<'a, CSSStyleDeclaration> { impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { fn Length(self) -> u32 { let owner = self.owner.root(); - let elem: JSRef<Element> = ElementCast::from_ref(*owner); + let elem: JSRef<Element> = ElementCast::from_ref(owner.r()); let len = match *elem.style_attribute().borrow() { Some(ref declarations) => declarations.normal.len() + declarations.important.len(), None => 0 @@ -109,7 +109,7 @@ impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { fn Item(self, index: u32) -> DOMString { let owner = self.owner.root(); - let elem: JSRef<Element> = ElementCast::from_ref(*owner); + let elem: JSRef<Element> = ElementCast::from_ref(owner.r()); let style_attribute = elem.style_attribute().borrow(); let result = style_attribute.as_ref().and_then(|declarations| { if index as uint > declarations.normal.len() { @@ -218,7 +218,7 @@ impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { synthesized_declaration.push_str(value.as_slice()); let owner = self.owner.root(); - let window = window_from_node(*owner).root(); + let window = window_from_node(owner.r()).root(); let page = window.page(); let decl_block = parse_style_attribute(synthesized_declaration.as_slice(), &page.get_url()); @@ -229,7 +229,7 @@ impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { } let owner = self.owner.root(); - let element: JSRef<Element> = ElementCast::from_ref(*owner); + let element: JSRef<Element> = ElementCast::from_ref(owner.r()); // Step 8 for decl in decl_block.normal.iter() { @@ -240,7 +240,7 @@ impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { let document = document_from_node(element).root(); let node: JSRef<Node> = NodeCast::from_ref(element); - document.content_changed(node, NodeDamage::NodeStyleDamaged); + document.r().content_changed(node, NodeDamage::NodeStyleDamaged); Ok(()) } @@ -266,11 +266,11 @@ impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { } let owner = self.owner.root(); - let window = window_from_node(*owner).root(); + let window = window_from_node(owner.r()).root(); let page = window.page(); let decl_block = parse_style_attribute(property.as_slice(), &page.get_url()); - let element: JSRef<Element> = ElementCast::from_ref(*owner); + let element: JSRef<Element> = ElementCast::from_ref(owner.r()); // Step 5 for decl in decl_block.normal.iter() { @@ -281,7 +281,7 @@ impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { let document = document_from_node(element).root(); let node: JSRef<Node> = NodeCast::from_ref(element); - document.content_changed(node, NodeDamage::NodeStyleDamaged); + document.r().content_changed(node, NodeDamage::NodeStyleDamaged); Ok(()) } @@ -315,7 +315,7 @@ impl<'a> CSSStyleDeclarationMethods for JSRef<'a, CSSStyleDeclaration> { None => { // Step 5 let owner = self.owner.root(); - let elem: JSRef<Element> = ElementCast::from_ref(*owner); + let elem: JSRef<Element> = ElementCast::from_ref(owner.r()); elem.remove_inline_style_property(property) } } diff --git a/components/script/dom/customevent.rs b/components/script/dom/customevent.rs index 9805cd57efa..249b4331d42 100644 --- a/components/script/dom/customevent.rs +++ b/components/script/dom/customevent.rs @@ -42,8 +42,8 @@ impl CustomEvent { } pub fn new(global: &GlobalRef, type_: DOMString, bubbles: bool, cancelable: bool, detail: JSVal) -> Temporary<CustomEvent> { let ev = CustomEvent::new_uninitialized(*global).root(); - ev.InitCustomEvent(global.get_cx(), type_, bubbles, cancelable, detail); - Temporary::from_rooted(*ev) + ev.r().InitCustomEvent(global.get_cx(), type_, bubbles, cancelable, detail); + Temporary::from_rooted(ev.r()) } pub fn Constructor(global: &GlobalRef, type_: DOMString, diff --git a/components/script/dom/dedicatedworkerglobalscope.rs b/components/script/dom/dedicatedworkerglobalscope.rs index 0044bd56254..b095cd7b104 100644 --- a/components/script/dom/dedicatedworkerglobalscope.rs +++ b/components/script/dom/dedicatedworkerglobalscope.rs @@ -158,20 +158,20 @@ impl DedicatedWorkerGlobalScope { parent_sender, own_sender, receiver).root(); { - let _ar = AutoWorkerReset::new(*global, worker); + let _ar = AutoWorkerReset::new(global.r(), worker); match js_context.evaluate_script( - global.reflector().get_jsobject(), source, url.serialize(), 1) { + global.r().reflector().get_jsobject(), source, url.serialize(), 1) { Ok(_) => (), Err(_) => println!("evaluate_script failed") } } loop { - match global.receiver.recv_opt() { + match global.r().receiver.recv_opt() { Ok((linked_worker, msg)) => { - let _ar = AutoWorkerReset::new(*global, linked_worker); - global.handle_event(msg); + let _ar = AutoWorkerReset::new(global.r(), linked_worker); + global.r().handle_event(msg); } Err(_) => break, } diff --git a/components/script/dom/document.rs b/components/script/dom/document.rs index f35a01b2419..0bf8070fe80 100644 --- a/components/script/dom/document.rs +++ b/components/script/dom/document.rs @@ -255,7 +255,7 @@ impl<'a> DocumentHelpers<'a> for JSRef<'a, Document> { Some(elements) => { let position = elements.iter() .map(|elem| elem.root()) - .position(|element| *element == to_unregister) + .position(|element| element.r() == to_unregister) .expect("This element should be in registered."); elements.remove(position); elements.is_empty() @@ -289,13 +289,13 @@ impl<'a> DocumentHelpers<'a> for JSRef<'a, Document> { let new_node: JSRef<Node> = NodeCast::from_ref(element); let mut head: uint = 0u; - let root: JSRef<Node> = NodeCast::from_ref(*root); + let root: JSRef<Node> = NodeCast::from_ref(root.r()); for node in root.traverse_preorder() { let elem: Option<JSRef<Element>> = ElementCast::to_ref(node); match elem { None => {}, Some(elem) => { - if *(*elements)[head].root() == elem { + if (*elements)[head].root().r() == elem { head += 1; } if new_node == node || head == elements.len() { @@ -312,7 +312,7 @@ impl<'a> DocumentHelpers<'a> for JSRef<'a, Document> { fn load_anchor_href(self, href: DOMString) { let window = self.window.root(); - window.load_url(href); + window.r().load_url(href); } /// Attempt to find a named element in this page's document. @@ -322,7 +322,7 @@ impl<'a> DocumentHelpers<'a> for JSRef<'a, Document> { let check_anchor = |&node: &JSRef<HTMLAnchorElement>| { let elem: JSRef<Element> = ElementCast::from_ref(node); elem.get_attribute(ns!(""), &atom!("name")).root().map_or(false, |attr| { - attr.value().as_slice() == fragid.as_slice() + attr.r().value().as_slice() == fragid.as_slice() }) }; let doc_node: JSRef<Node> = NodeCast::from_ref(self); @@ -338,11 +338,11 @@ impl<'a> DocumentHelpers<'a> for JSRef<'a, Document> { self.ready_state.set(state); let window = self.window.root(); - let event = Event::new(GlobalRef::Window(*window), "readystatechange".into_string(), + let event = Event::new(GlobalRef::Window(window.r()), "readystatechange".into_string(), EventBubbles::DoesNotBubble, EventCancelable::NotCancelable).root(); let target: JSRef<EventTarget> = EventTargetCast::from_ref(self); - let _ = target.DispatchEvent(*event); + let _ = target.DispatchEvent(event.r()); } /// Return the element that currently has focus. @@ -372,7 +372,7 @@ impl<'a> DocumentHelpers<'a> for JSRef<'a, Document> { /// Sends this document's title to the compositor. fn send_title_to_compositor(self) { let window = self.window().root(); - window.page().send_title_to_compositor(); + window.r().page().send_title_to_compositor(); } fn dirty_all_nodes(self) { @@ -466,9 +466,9 @@ impl Document { GlobalRef::Window(window), DocumentBinding::Wrap).root(); - let node: JSRef<Node> = NodeCast::from_ref(*document); - node.set_owner_doc(*document); - Temporary::from_rooted(*document) + let node: JSRef<Node> = NodeCast::from_ref(document.r()); + node.set_owner_doc(document.r()); + Temporary::from_rooted(document.r()) } } @@ -487,7 +487,7 @@ impl<'a> PrivateDocumentHelpers for JSRef<'a, Document> { root.traverse_preorder().filter(|&node| callback(node)).collect() } }; - NodeList::new_simple_list(*window, nodes) + NodeList::new_simple_list(window.r(), nodes) } fn get_html_element(self) -> Option<Temporary<HTMLHtmlElement>> { @@ -554,20 +554,20 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { // http://dom.spec.whatwg.org/#dom-document-getelementsbytagname fn GetElementsByTagName(self, tag_name: DOMString) -> Temporary<HTMLCollection> { let window = self.window.root(); - HTMLCollection::by_tag_name(*window, NodeCast::from_ref(self), tag_name) + HTMLCollection::by_tag_name(window.r(), NodeCast::from_ref(self), tag_name) } // http://dom.spec.whatwg.org/#dom-document-getelementsbytagnamens fn GetElementsByTagNameNS(self, maybe_ns: Option<DOMString>, tag_name: DOMString) -> Temporary<HTMLCollection> { let window = self.window.root(); - HTMLCollection::by_tag_name_ns(*window, NodeCast::from_ref(self), tag_name, maybe_ns) + HTMLCollection::by_tag_name_ns(window.r(), NodeCast::from_ref(self), tag_name, maybe_ns) } // http://dom.spec.whatwg.org/#dom-document-getelementsbyclassname fn GetElementsByClassName(self, classes: DOMString) -> Temporary<HTMLCollection> { let window = self.window.root(); - HTMLCollection::by_class_name(*window, NodeCast::from_ref(self), classes) + HTMLCollection::by_class_name(window.r(), NodeCast::from_ref(self), classes) } // http://dom.spec.whatwg.org/#dom-nonelementparentnode-getelementbyid @@ -652,7 +652,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let l_name = Atom::from_slice(local_name.as_slice()); let value = AttrValue::String("".into_string()); - Ok(Attr::new(*window, name, value, l_name, ns!(""), None, None)) + Ok(Attr::new(window.r(), name, value, l_name, ns!(""), None, None)) } // http://dom.spec.whatwg.org/#dom-document-createdocumentfragment @@ -724,17 +724,17 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { match interface.as_slice().to_ascii_lower().as_slice() { "uievents" | "uievent" => Ok(EventCast::from_temporary( - UIEvent::new_uninitialized(*window))), + UIEvent::new_uninitialized(window.r()))), "mouseevents" | "mouseevent" => Ok(EventCast::from_temporary( - MouseEvent::new_uninitialized(*window))), + MouseEvent::new_uninitialized(window.r()))), "customevent" => Ok(EventCast::from_temporary( - CustomEvent::new_uninitialized(GlobalRef::Window(*window)))), + CustomEvent::new_uninitialized(GlobalRef::Window(window.r())))), "htmlevents" | "events" | "event" => Ok(Event::new_uninitialized( - GlobalRef::Window(*window))), + GlobalRef::Window(window.r()))), "keyboardevent" | "keyevents" => Ok(EventCast::from_temporary( - KeyboardEvent::new_uninitialized(*window))), + KeyboardEvent::new_uninitialized(window.r()))), "messageevent" => Ok(EventCast::from_temporary( - MessageEvent::new_uninitialized(GlobalRef::Window(*window)))), + MessageEvent::new_uninitialized(GlobalRef::Window(window.r())))), _ => Err(NotSupported) } } @@ -762,7 +762,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { fn Title(self) -> DOMString { let mut title = String::new(); self.GetDocumentElement().root().map(|root| { - let root: JSRef<Node> = NodeCast::from_ref(*root); + let root: JSRef<Node> = NodeCast::from_ref(root.r()); root.traverse_preorder() .find(|node| node.type_id() == NodeTypeId::Element(ElementTypeId::HTMLTitleElement)) .map(|title_elem| { @@ -778,7 +778,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { // http://www.whatwg.org/specs/web-apps/current-work/#document.title fn SetTitle(self, title: DOMString) -> ErrorResult { self.GetDocumentElement().root().map(|root| { - let root: JSRef<Node> = NodeCast::from_ref(*root); + let root: JSRef<Node> = NodeCast::from_ref(root.r()); let head_node = root.traverse_preorder().find(|child| { child.type_id() == NodeTypeId::Element(ElementTypeId::HTMLHeadElement) }); @@ -794,16 +794,16 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { } if !title.is_empty() { let new_text = self.CreateTextNode(title.clone()).root(); - assert!(title_node.AppendChild(NodeCast::from_ref(*new_text)).is_ok()); + assert!(title_node.AppendChild(NodeCast::from_ref(new_text.r())).is_ok()); } }, None => { let new_title = HTMLTitleElement::new("title".into_string(), None, self).root(); - let new_title: JSRef<Node> = NodeCast::from_ref(*new_title); + let new_title: JSRef<Node> = NodeCast::from_ref(new_title.r()); if !title.is_empty() { let new_text = self.CreateTextNode(title.clone()).root(); - assert!(new_title.AppendChild(NodeCast::from_ref(*new_text)).is_ok()); + assert!(new_title.AppendChild(NodeCast::from_ref(new_text.r())).is_ok()); } assert!(head.AppendChild(new_title).is_ok()); }, @@ -817,7 +817,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { fn GetHead(self) -> Option<Temporary<HTMLHeadElement>> { self.get_html_element().and_then(|root| { let root = root.root(); - let node: JSRef<Node> = NodeCast::from_ref(*root); + let node: JSRef<Node> = NodeCast::from_ref(root.r()); node.children().filter_map(HTMLHeadElementCast::to_ref).next().map(Temporary::from_rooted) }) } @@ -826,7 +826,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { fn GetBody(self) -> Option<Temporary<HTMLElement>> { self.get_html_element().and_then(|root| { let root = root.root(); - let node: JSRef<Node> = NodeCast::from_ref(*root); + let node: JSRef<Node> = NodeCast::from_ref(root.r()); node.children().find(|child| { match child.type_id() { NodeTypeId::Element(ElementTypeId::HTMLBodyElement) | @@ -856,7 +856,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { // Step 2. let old_body = self.GetBody().root(); - if old_body.as_ref().map(|body| **body) == Some(new_body) { + if old_body.as_ref().map(|body| body.r()) == Some(new_body) { return Ok(()); } @@ -867,10 +867,10 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { Some(ref root) => { let new_body: JSRef<Node> = NodeCast::from_ref(new_body); - let root: JSRef<Node> = NodeCast::from_ref(**root); + let root: JSRef<Node> = NodeCast::from_ref(root.r()); match old_body { Some(ref child) => { - let child: JSRef<Node> = NodeCast::from_ref(**child); + let child: JSRef<Node> = NodeCast::from_ref(child.r()); assert!(root.ReplaceChild(new_body, child).is_ok()) } @@ -889,7 +889,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { None => return false, }; element.get_attribute(ns!(""), &atom!("name")).root().map_or(false, |attr| { - attr.value().as_slice() == name.as_slice() + attr.r().value().as_slice() == name.as_slice() }) }) } @@ -899,7 +899,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let window = self.window.root(); let root = NodeCast::from_ref(self); let filter = box ImagesFilter; - HTMLCollection::create(*window, root, filter) + HTMLCollection::create(window.r(), root, filter) }) } @@ -908,7 +908,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let window = self.window.root(); let root = NodeCast::from_ref(self); let filter = box EmbedsFilter; - HTMLCollection::create(*window, root, filter) + HTMLCollection::create(window.r(), root, filter) }) } @@ -921,7 +921,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let window = self.window.root(); let root = NodeCast::from_ref(self); let filter = box LinksFilter; - HTMLCollection::create(*window, root, filter) + HTMLCollection::create(window.r(), root, filter) }) } @@ -930,7 +930,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let window = self.window.root(); let root = NodeCast::from_ref(self); let filter = box FormsFilter; - HTMLCollection::create(*window, root, filter) + HTMLCollection::create(window.r(), root, filter) }) } @@ -939,7 +939,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let window = self.window.root(); let root = NodeCast::from_ref(self); let filter = box ScriptsFilter; - HTMLCollection::create(*window, root, filter) + HTMLCollection::create(window.r(), root, filter) }) } @@ -948,7 +948,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let window = self.window.root(); let root = NodeCast::from_ref(self); let filter = box AnchorsFilter; - HTMLCollection::create(*window, root, filter) + HTMLCollection::create(window.r(), root, filter) }) } @@ -958,19 +958,19 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let window = self.window.root(); let root = NodeCast::from_ref(self); let filter = box AppletsFilter; - HTMLCollection::create(*window, root, filter) + HTMLCollection::create(window.r(), root, filter) }) } fn Location(self) -> Temporary<Location> { let window = self.window.root(); - window.Location() + window.r().Location() } // http://dom.spec.whatwg.org/#dom-parentnode-children fn Children(self) -> Temporary<HTMLCollection> { let window = self.window.root(); - HTMLCollection::children(*window, NodeCast::from_ref(self)) + HTMLCollection::children(window.r(), NodeCast::from_ref(self)) } // http://dom.spec.whatwg.org/#dom-parentnode-queryselector diff --git a/components/script/dom/documentfragment.rs b/components/script/dom/documentfragment.rs index fdda82beced..7c35446becd 100644 --- a/components/script/dom/documentfragment.rs +++ b/components/script/dom/documentfragment.rs @@ -45,7 +45,7 @@ impl DocumentFragment { let document = global.as_window().Document(); let document = document.root(); - Ok(DocumentFragment::new(*document)) + Ok(DocumentFragment::new(document.r())) } } @@ -53,7 +53,7 @@ impl<'a> DocumentFragmentMethods for JSRef<'a, DocumentFragment> { // http://dom.spec.whatwg.org/#dom-parentnode-children fn Children(self) -> Temporary<HTMLCollection> { let window = window_from_node(self).root(); - HTMLCollection::children(*window, NodeCast::from_ref(self)) + HTMLCollection::children(window.r(), NodeCast::from_ref(self)) } // http://dom.spec.whatwg.org/#dom-parentnode-queryselector diff --git a/components/script/dom/domimplementation.rs b/components/script/dom/domimplementation.rs index b2550f0d642..e298d275f47 100644 --- a/components/script/dom/domimplementation.rs +++ b/components/script/dom/domimplementation.rs @@ -42,7 +42,7 @@ impl DOMImplementation { pub fn new(document: JSRef<Document>) -> Temporary<DOMImplementation> { let window = document.window().root(); reflect_dom_object(box DOMImplementation::new_inherited(document), - GlobalRef::Window(*window), + GlobalRef::Window(window.r()), DOMImplementationBinding::Wrap) } } @@ -59,7 +59,7 @@ impl<'a> DOMImplementationMethods for JSRef<'a, DOMImplementation> { // Step 3. QName => { let document = self.document.root(); - Ok(DocumentType::new(qname, Some(pubid), Some(sysid), *document)) + Ok(DocumentType::new(qname, Some(pubid), Some(sysid), document.r())) } } } @@ -68,23 +68,23 @@ impl<'a> DOMImplementationMethods for JSRef<'a, DOMImplementation> { fn CreateDocument(self, namespace: Option<DOMString>, qname: DOMString, maybe_doctype: Option<JSRef<DocumentType>>) -> Fallible<Temporary<Document>> { let doc = self.document.root(); - let win = doc.window().root(); + let win = doc.r().window().root(); // Step 1. - let doc = Document::new(*win, None, IsHTMLDocument::NonHTMLDocument, + let doc = Document::new(win.r(), None, IsHTMLDocument::NonHTMLDocument, None, DocumentSource::NotFromParser).root(); // Step 2-3. let maybe_elem = if qname.is_empty() { None } else { - match doc.CreateElementNS(namespace, qname) { + match doc.r().CreateElementNS(namespace, qname) { Err(error) => return Err(error), Ok(elem) => Some(elem) } }; { - let doc_node: JSRef<Node> = NodeCast::from_ref(*doc); + let doc_node: JSRef<Node> = NodeCast::from_ref(doc.r()); // Step 4. match maybe_doctype { @@ -99,7 +99,7 @@ impl<'a> DOMImplementationMethods for JSRef<'a, DOMImplementation> { match maybe_elem.root() { None => (), Some(elem) => { - assert!(doc_node.AppendChild(NodeCast::from_ref(*elem)).is_ok()) + assert!(doc_node.AppendChild(NodeCast::from_ref(elem.r())).is_ok()) } } } @@ -108,60 +108,60 @@ impl<'a> DOMImplementationMethods for JSRef<'a, DOMImplementation> { // FIXME: https://github.com/mozilla/servo/issues/1522 // Step 7. - Ok(Temporary::from_rooted(*doc)) + Ok(Temporary::from_rooted(doc.r())) } // http://dom.spec.whatwg.org/#dom-domimplementation-createhtmldocument fn CreateHTMLDocument(self, title: Option<DOMString>) -> Temporary<Document> { let document = self.document.root(); - let win = document.window().root(); + let win = document.r().window().root(); // Step 1-2. - let doc = Document::new(*win, None, IsHTMLDocument::HTMLDocument, None, + let doc = Document::new(win.r(), None, IsHTMLDocument::HTMLDocument, None, DocumentSource::NotFromParser).root(); - let doc_node: JSRef<Node> = NodeCast::from_ref(*doc); + let doc_node: JSRef<Node> = NodeCast::from_ref(doc.r()); { // Step 3. - let doc_type = DocumentType::new("html".into_string(), None, None, *doc).root(); - assert!(doc_node.AppendChild(NodeCast::from_ref(*doc_type)).is_ok()); + let doc_type = DocumentType::new("html".into_string(), None, None, doc.r()).root(); + assert!(doc_node.AppendChild(NodeCast::from_ref(doc_type.r())).is_ok()); } { // Step 4. - let doc_html: Root<Node> = NodeCast::from_temporary(HTMLHtmlElement::new("html".into_string(), None, *doc)).root(); - assert!(doc_node.AppendChild(*doc_html).is_ok()); + let doc_html: Root<Node> = NodeCast::from_temporary(HTMLHtmlElement::new("html".into_string(), None, doc.r())).root(); + assert!(doc_node.AppendChild(doc_html.r()).is_ok()); { // Step 5. - let doc_head: Root<Node> = NodeCast::from_temporary(HTMLHeadElement::new("head".into_string(), None, *doc)).root(); - assert!(doc_html.AppendChild(*doc_head).is_ok()); + let doc_head: Root<Node> = NodeCast::from_temporary(HTMLHeadElement::new("head".into_string(), None, doc.r())).root(); + assert!(doc_html.r().AppendChild(doc_head.r()).is_ok()); // Step 6. match title { None => (), Some(title_str) => { // Step 6.1. - let doc_title: Root<Node> = NodeCast::from_temporary(HTMLTitleElement::new("title".into_string(), None, *doc)).root(); - assert!(doc_head.AppendChild(*doc_title).is_ok()); + let doc_title: Root<Node> = NodeCast::from_temporary(HTMLTitleElement::new("title".into_string(), None, doc.r())).root(); + assert!(doc_head.r().AppendChild(doc_title.r()).is_ok()); // Step 6.2. - let title_text: Root<Text> = Text::new(title_str, *doc).root(); - assert!(doc_title.AppendChild(NodeCast::from_ref(*title_text)).is_ok()); + let title_text: Root<Text> = Text::new(title_str, doc.r()).root(); + assert!(doc_title.r().AppendChild(NodeCast::from_ref(title_text.r())).is_ok()); } } } // Step 7. - let doc_body: Root<HTMLBodyElement> = HTMLBodyElement::new("body".into_string(), None, *doc).root(); - assert!(doc_html.AppendChild(NodeCast::from_ref(*doc_body)).is_ok()); + let doc_body: Root<HTMLBodyElement> = HTMLBodyElement::new("body".into_string(), None, doc.r()).root(); + assert!(doc_html.r().AppendChild(NodeCast::from_ref(doc_body.r())).is_ok()); } // Step 8. // FIXME: https://github.com/mozilla/servo/issues/1522 // Step 9. - Temporary::from_rooted(*doc) + Temporary::from_rooted(doc.r()) } // https://dom.spec.whatwg.org/#dom-domimplementation-hasfeature diff --git a/components/script/dom/domstringmap.rs b/components/script/dom/domstringmap.rs index 268087a3533..6b2b4fafffb 100644 --- a/components/script/dom/domstringmap.rs +++ b/components/script/dom/domstringmap.rs @@ -41,17 +41,17 @@ impl<'a> DOMStringMapMethods for JSRef<'a, DOMStringMap> { fn NamedDeleter(self, name: DOMString) { let element = self.element.root(); - element.delete_custom_attr(name) + element.r().delete_custom_attr(name) } fn NamedSetter(self, name: DOMString, value: DOMString) -> ErrorResult { let element = self.element.root(); - element.set_custom_attr(name, value) + element.r().set_custom_attr(name, value) } fn NamedGetter(self, name: DOMString, found: &mut bool) -> DOMString { let element = self.element.root(); - match element.get_custom_attr(name) { + match element.r().get_custom_attr(name) { Some(value) => { *found = true; value.clone() diff --git a/components/script/dom/domtokenlist.rs b/components/script/dom/domtokenlist.rs index 8edf6f18c3c..c681dc7d710 100644 --- a/components/script/dom/domtokenlist.rs +++ b/components/script/dom/domtokenlist.rs @@ -35,7 +35,7 @@ impl DOMTokenList { pub fn new(element: JSRef<Element>, local_name: &Atom) -> Temporary<DOMTokenList> { let window = window_from_node(element).root(); reflect_dom_object(box DOMTokenList::new_inherited(element, local_name.clone()), - GlobalRef::Window(*window), + GlobalRef::Window(window.r()), DOMTokenListBinding::Wrap) } } @@ -48,7 +48,7 @@ trait PrivateDOMTokenListHelpers { impl<'a> PrivateDOMTokenListHelpers for JSRef<'a, DOMTokenList> { fn attribute(self) -> Option<Temporary<Attr>> { let element = self.element.root(); - element.get_attribute(ns!(""), &self.local_name) + element.r().get_attribute(ns!(""), &self.local_name) } fn check_token_exceptions<'a>(self, token: &'a str) -> Fallible<Atom> { @@ -65,13 +65,13 @@ impl<'a> DOMTokenListMethods for JSRef<'a, DOMTokenList> { // http://dom.spec.whatwg.org/#dom-domtokenlist-length fn Length(self) -> u32 { self.attribute().root().map(|attr| { - attr.value().tokens().map(|tokens| tokens.len()).unwrap_or(0) + attr.r().value().tokens().map(|tokens| tokens.len()).unwrap_or(0) }).unwrap_or(0) as u32 } // http://dom.spec.whatwg.org/#dom-domtokenlist-item fn Item(self, index: u32) -> Option<DOMString> { - self.attribute().root().and_then(|attr| attr.value().tokens().and_then(|tokens| { + self.attribute().root().and_then(|attr| attr.r().value().tokens().and_then(|tokens| { tokens.get(index as uint).map(|token| token.as_slice().into_string()) })) } @@ -86,7 +86,8 @@ impl<'a> DOMTokenListMethods for JSRef<'a, DOMTokenList> { fn Contains(self, token: DOMString) -> Fallible<bool> { self.check_token_exceptions(token.as_slice()).map(|token| { self.attribute().root().map(|attr| { - attr.value() + attr.r() + .value() .tokens() .expect("Should have parsed this attribute") .iter() @@ -98,42 +99,42 @@ impl<'a> DOMTokenListMethods for JSRef<'a, DOMTokenList> { // https://dom.spec.whatwg.org/#dom-domtokenlist-add fn Add(self, tokens: Vec<DOMString>) -> ErrorResult { let element = self.element.root(); - let mut atoms = element.get_tokenlist_attribute(&self.local_name); + let mut atoms = element.r().get_tokenlist_attribute(&self.local_name); for token in tokens.iter() { let token = try!(self.check_token_exceptions(token.as_slice())); if !atoms.iter().any(|atom| *atom == token) { atoms.push(token); } } - element.set_atomic_tokenlist_attribute(&self.local_name, atoms); + element.r().set_atomic_tokenlist_attribute(&self.local_name, atoms); Ok(()) } // https://dom.spec.whatwg.org/#dom-domtokenlist-remove fn Remove(self, tokens: Vec<DOMString>) -> ErrorResult { let element = self.element.root(); - let mut atoms = element.get_tokenlist_attribute(&self.local_name); + let mut atoms = element.r().get_tokenlist_attribute(&self.local_name); for token in tokens.iter() { let token = try!(self.check_token_exceptions(token.as_slice())); atoms.iter().position(|atom| *atom == token).and_then(|index| { atoms.remove(index) }); } - element.set_atomic_tokenlist_attribute(&self.local_name, atoms); + element.r().set_atomic_tokenlist_attribute(&self.local_name, atoms); Ok(()) } // https://dom.spec.whatwg.org/#dom-domtokenlist-toggle fn Toggle(self, token: DOMString, force: Option<bool>) -> Fallible<bool> { let element = self.element.root(); - let mut atoms = element.get_tokenlist_attribute(&self.local_name); + let mut atoms = element.r().get_tokenlist_attribute(&self.local_name); let token = try!(self.check_token_exceptions(token.as_slice())); match atoms.iter().position(|atom| *atom == token) { Some(index) => match force { Some(true) => Ok(true), _ => { atoms.remove(index); - element.set_atomic_tokenlist_attribute(&self.local_name, atoms); + element.r().set_atomic_tokenlist_attribute(&self.local_name, atoms); Ok(false) } }, @@ -141,7 +142,7 @@ impl<'a> DOMTokenListMethods for JSRef<'a, DOMTokenList> { Some(false) => Ok(false), _ => { atoms.push(token); - element.set_atomic_tokenlist_attribute(&self.local_name, atoms); + element.r().set_atomic_tokenlist_attribute(&self.local_name, atoms); Ok(true) } } diff --git a/components/script/dom/element.rs b/components/script/dom/element.rs index a452eafc6fb..77de0380c98 100644 --- a/components/script/dom/element.rs +++ b/components/script/dom/element.rs @@ -515,9 +515,9 @@ impl<'a> ElementHelpers<'a> for JSRef<'a, Element> { let attrs = self.Attributes().root(); let mut i = 0; let mut summarized = vec!(); - while i < attrs.Length() { - let attr = attrs.Item(i).unwrap().root(); - summarized.push(attr.summarize()); + while i < attrs.r().Length() { + let attr = attrs.r().Item(i).unwrap().root(); + summarized.push(attr.r().summarize()); i += 1; } summarized @@ -658,14 +658,14 @@ pub trait AttributeHandlers { impl<'a> AttributeHandlers for JSRef<'a, Element> { fn get_attribute(self, namespace: Namespace, local_name: &Atom) -> Option<Temporary<Attr>> { self.get_attributes(local_name).iter().map(|attr| attr.root()) - .find(|attr| *attr.namespace() == namespace) - .map(|x| Temporary::from_rooted(*x)) + .find(|attr| *attr.r().namespace() == namespace) + .map(|x| Temporary::from_rooted(x.r())) } fn get_attributes(self, local_name: &Atom) -> Vec<Temporary<Attr>> { self.attrs.borrow().iter().map(|attr| attr.root()).filter_map(|attr| { - if *attr.local_name() == *local_name { - Some(Temporary::from_rooted(*attr)) + if *attr.r().local_name() == *local_name { + Some(Temporary::from_rooted(attr.r())) } else { None } @@ -678,7 +678,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { prefix: Option<DOMString>) { // Don't set if the attribute already exists, so we can handle add_attrs_if_missing if self.attrs.borrow().iter().map(|attr| attr.root()) - .any(|a| *a.local_name() == qname.local && *a.namespace() == qname.ns) { + .any(|a| *a.r().local_name() == qname.local && *a.r().namespace() == qname.ns) { return; } @@ -723,19 +723,19 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { prefix: Option<DOMString>, cb: |JSRef<Attr>| -> bool) { let idx = self.attrs.borrow().iter() .map(|attr| attr.root()) - .position(|attr| cb(*attr)); + .position(|attr| cb(attr.r())); let (idx, set_type) = match idx { Some(idx) => (idx, AttrSettingType::ReplacedAttr), None => { let window = window_from_node(self).root(); - let attr = Attr::new(*window, local_name, value.clone(), + let attr = Attr::new(window.r(), local_name, value.clone(), name, namespace.clone(), prefix, Some(self)); self.attrs.borrow_mut().push_unrooted(&attr); (self.attrs.borrow().len() - 1, AttrSettingType::FirstSetAttr) } }; - (*self.attrs.borrow())[idx].root().set_value(set_type, value, self); + (*self.attrs.borrow())[idx].root().r().set_value(set_type, value, self); } fn parse_attribute(self, namespace: &Namespace, local_name: &Atom, @@ -753,7 +753,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { let local_name = Atom::from_slice(local_name); let idx = self.attrs.borrow().iter().map(|attr| attr.root()).position(|attr| { - *attr.local_name() == local_name + *attr.r().local_name() == local_name }); match idx { @@ -761,7 +761,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { Some(idx) => { if namespace == ns!("") { let attr = (*self.attrs.borrow())[idx].root(); - vtable_for(&NodeCast::from_ref(self)).before_remove_attr(*attr); + vtable_for(&NodeCast::from_ref(self)).before_remove_attr(attr.r()); } self.attrs.borrow_mut().remove(idx); @@ -770,9 +770,9 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { if node.is_in_doc() { let document = document_from_node(self).root(); if local_name == atom!("style") { - document.content_changed(node, NodeDamage::NodeStyleDamaged); + document.r().content_changed(node, NodeDamage::NodeStyleDamaged); } else { - document.content_changed(node, NodeDamage::OtherNodeDamage); + document.r().content_changed(node, NodeDamage::OtherNodeDamage); } } } @@ -781,7 +781,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { fn has_class(&self, name: &Atom) -> bool { self.get_attribute(ns!(""), &atom!("class")).root().map(|attr| { - attr.value().tokens().map(|tokens| { + attr.r().value().tokens().map(|tokens| { tokens.iter().any(|atom| atom == name) }).unwrap_or(false) }).unwrap_or(false) @@ -798,7 +798,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { !ch.is_ascii() || ch.to_ascii().to_lowercase() == ch.to_ascii() })); self.attrs.borrow().iter().map(|attr| attr.root()).any(|attr| { - *attr.local_name() == *name && *attr.namespace() == ns!("") + *attr.r().local_name() == *name && *attr.r().namespace() == ns!("") }) } @@ -818,7 +818,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { } let url = self.get_string_attribute(name); let doc = document_from_node(self).root(); - let base = doc.url(); + let base = doc.r().url(); // https://html.spec.whatwg.org/multipage/infrastructure.html#reflect // XXXManishearth this doesn't handle `javascript:` urls properly match UrlParser::new().base_url(base).parse(url.as_slice()) { @@ -832,7 +832,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { fn get_string_attribute(self, name: &Atom) -> DOMString { match self.get_attribute(ns!(""), name) { - Some(x) => x.root().Value(), + Some(x) => x.root().r().Value(), None => "".into_string() } } @@ -843,7 +843,8 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { fn get_tokenlist_attribute(self, name: &Atom) -> Vec<Atom> { self.get_attribute(ns!(""), name).root().map(|attr| { - attr.value() + attr.r() + .value() .tokens() .expect("Expected a TokenListAttrValue") .to_vec() @@ -867,7 +868,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { let attribute = self.get_attribute(ns!(""), name).root(); match attribute { Some(attribute) => { - match *attribute.value() { + match *attribute.r().value() { AttrValue::UInt(_, value) => value, _ => panic!("Expected an AttrValue::UInt: \ implement parse_plain_attribute"), @@ -949,8 +950,8 @@ impl<'a> ElementMethods for JSRef<'a, Element> { let node: JSRef<Node> = NodeCast::from_ref(self); node.owner_doc().root() }; - let window = doc.window().root(); - NamedNodeMap::new(*window, self) + let window = doc.r().window().root(); + NamedNodeMap::new(window.r(), self) }) } @@ -958,7 +959,7 @@ impl<'a> ElementMethods for JSRef<'a, Element> { fn GetAttribute(self, name: DOMString) -> Option<DOMString> { let name = self.parsed_name(name); self.get_attribute(ns!(""), &Atom::from_slice(name.as_slice())).root() - .map(|s| s.Value()) + .map(|s| s.r().Value()) } // http://dom.spec.whatwg.org/#dom-element-getattributens @@ -967,7 +968,7 @@ impl<'a> ElementMethods for JSRef<'a, Element> { local_name: DOMString) -> Option<DOMString> { let namespace = namespace::from_domstring(namespace); self.get_attribute(namespace, &Atom::from_slice(local_name.as_slice())).root() - .map(|attr| attr.Value()) + .map(|attr| attr.r().Value()) } // http://dom.spec.whatwg.org/#dom-element-setattribute @@ -1084,18 +1085,18 @@ impl<'a> ElementMethods for JSRef<'a, Element> { fn GetElementsByTagName(self, localname: DOMString) -> Temporary<HTMLCollection> { let window = window_from_node(self).root(); - HTMLCollection::by_tag_name(*window, NodeCast::from_ref(self), localname) + HTMLCollection::by_tag_name(window.r(), NodeCast::from_ref(self), localname) } fn GetElementsByTagNameNS(self, maybe_ns: Option<DOMString>, localname: DOMString) -> Temporary<HTMLCollection> { let window = window_from_node(self).root(); - HTMLCollection::by_tag_name_ns(*window, NodeCast::from_ref(self), localname, maybe_ns) + HTMLCollection::by_tag_name_ns(window.r(), NodeCast::from_ref(self), localname, maybe_ns) } fn GetElementsByClassName(self, classes: DOMString) -> Temporary<HTMLCollection> { let window = window_from_node(self).root(); - HTMLCollection::by_class_name(*window, NodeCast::from_ref(self), classes) + HTMLCollection::by_class_name(window.r(), NodeCast::from_ref(self), classes) } // http://dev.w3.org/csswg/cssom-view/#dom-element-getclientrects @@ -1105,14 +1106,14 @@ impl<'a> ElementMethods for JSRef<'a, Element> { let rects = node.get_content_boxes(); let rects: Vec<Root<DOMRect>> = rects.iter().map(|r| { DOMRect::new( - *win, + win.r(), r.origin.y, r.origin.y + r.size.height, r.origin.x, r.origin.x + r.size.width).root() }).collect(); - DOMRectList::new(*win, rects.iter().map(|rect| rect.deref().clone()).collect()) + DOMRectList::new(win.r(), rects.iter().map(|rect| rect.r()).collect()) } // http://dev.w3.org/csswg/cssom-view/#dom-element-getboundingclientrect @@ -1121,7 +1122,7 @@ impl<'a> ElementMethods for JSRef<'a, Element> { let node: JSRef<Node> = NodeCast::from_ref(self); let rect = node.get_bounding_content_box(); DOMRect::new( - *win, + win.r(), rect.origin.y, rect.origin.y + rect.size.height, rect.origin.x, @@ -1140,7 +1141,7 @@ impl<'a> ElementMethods for JSRef<'a, Element> { // http://dom.spec.whatwg.org/#dom-parentnode-children fn Children(self) -> Temporary<HTMLCollection> { let window = window_from_node(self).root(); - HTMLCollection::children(*window, NodeCast::from_ref(self)) + HTMLCollection::children(window.r(), NodeCast::from_ref(self)) } // http://dom.spec.whatwg.org/#dom-parentnode-queryselector @@ -1206,13 +1207,13 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { // Modifying the `style` attribute might change style. let node: JSRef<Node> = NodeCast::from_ref(*self); let doc = document_from_node(*self).root(); - let base_url = doc.url().clone(); + let base_url = doc.r().url().clone(); let value = attr.value(); let style = Some(style::parse_style_attribute(value.as_slice(), &base_url)); *self.style_attribute.borrow_mut() = style; if node.is_in_doc() { - doc.content_changed(node, NodeDamage::NodeStyleDamaged); + doc.r().content_changed(node, NodeDamage::NodeStyleDamaged); } } &atom!("class") => { @@ -1220,7 +1221,7 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { let node: JSRef<Node> = NodeCast::from_ref(*self); if node.is_in_doc() { let document = document_from_node(*self).root(); - document.content_changed(node, NodeDamage::NodeStyleDamaged); + document.r().content_changed(node, NodeDamage::NodeStyleDamaged); } } &atom!("id") => { @@ -1231,9 +1232,9 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { let doc = document_from_node(*self).root(); if !value.as_slice().is_empty() { let value = Atom::from_slice(value.as_slice()); - doc.register_named_element(*self, value); + doc.r().register_named_element(*self, value); } - doc.content_changed(node, NodeDamage::NodeStyleDamaged); + doc.r().content_changed(node, NodeDamage::NodeStyleDamaged); } } _ => { @@ -1241,7 +1242,7 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { let node: JSRef<Node> = NodeCast::from_ref(*self); if node.is_in_doc() { let document = document_from_node(*self).root(); - document.content_changed(node, NodeDamage::OtherNodeDamage); + document.r().content_changed(node, NodeDamage::OtherNodeDamage); } } } @@ -1261,7 +1262,7 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { let node: JSRef<Node> = NodeCast::from_ref(*self); if node.is_in_doc() { let doc = document_from_node(*self).root(); - doc.content_changed(node, NodeDamage::NodeStyleDamaged); + doc.r().content_changed(node, NodeDamage::NodeStyleDamaged); } } &atom!("id") => { @@ -1272,9 +1273,9 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { let doc = document_from_node(*self).root(); if !value.as_slice().is_empty() { let value = Atom::from_slice(value.as_slice()); - doc.unregister_named_element(*self, value); + doc.r().unregister_named_element(*self, value); } - doc.content_changed(node, NodeDamage::NodeStyleDamaged); + doc.r().content_changed(node, NodeDamage::NodeStyleDamaged); } } &atom!("class") => { @@ -1282,7 +1283,7 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { let node: JSRef<Node> = NodeCast::from_ref(*self); if node.is_in_doc() { let document = document_from_node(*self).root(); - document.content_changed(node, NodeDamage::NodeStyleDamaged); + document.r().content_changed(node, NodeDamage::NodeStyleDamaged); } } _ => { @@ -1290,7 +1291,7 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { let node: JSRef<Node> = NodeCast::from_ref(*self); if node.is_in_doc() { let doc = document_from_node(*self).root(); - doc.content_changed(node, NodeDamage::OtherNodeDamage); + doc.r().content_changed(node, NodeDamage::OtherNodeDamage); } } } @@ -1315,10 +1316,10 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { match self.get_attribute(ns!(""), &atom!("id")).root() { Some(attr) => { let doc = document_from_node(*self).root(); - let value = attr.Value(); + let value = attr.r().Value(); if !value.is_empty() { let value = Atom::from_slice(value.as_slice()); - doc.register_named_element(*self, value); + doc.r().register_named_element(*self, value); } } _ => () @@ -1336,10 +1337,10 @@ impl<'a> VirtualMethods for JSRef<'a, Element> { match self.get_attribute(ns!(""), &atom!("id")).root() { Some(attr) => { let doc = document_from_node(*self).root(); - let value = attr.Value(); + let value = attr.r().Value(); if !value.is_empty() { let value = Atom::from_slice(value.as_slice()); - doc.unregister_named_element(*self, value); + doc.r().unregister_named_element(*self, value); } } _ => () @@ -1351,13 +1352,13 @@ impl<'a> style::TElement<'a> for JSRef<'a, Element> { fn get_attr(self, namespace: &Namespace, attr: &Atom) -> Option<&'a str> { self.get_attribute(namespace.clone(), attr).root().map(|attr| { // This transmute is used to cheat the lifetime restriction. - unsafe { mem::transmute(attr.value().as_slice()) } + unsafe { mem::transmute(attr.r().value().as_slice()) } }) } fn get_attrs(self, attr: &Atom) -> Vec<&'a str> { self.get_attributes(attr).iter().map(|attr| attr.root()).map(|attr| { // This transmute is used to cheat the lifetime restriction. - unsafe { mem::transmute(attr.value().as_slice()) } + unsafe { mem::transmute(attr.r().value().as_slice()) } }).collect() } fn get_link(self) -> Option<&'a str> { @@ -1397,7 +1398,7 @@ impl<'a> style::TElement<'a> for JSRef<'a, Element> { fn get_id(self) -> Option<Atom> { self.get_attribute(ns!(""), &atom!("id")).map(|attr| { let attr = attr.root(); - match *attr.value() { + match *attr.r().value() { AttrValue::Atom(ref val) => val.clone(), _ => panic!("`id` attribute should be AttrValue::Atom"), } @@ -1436,7 +1437,7 @@ impl<'a> style::TElement<'a> for JSRef<'a, Element> { match self.get_attribute(ns!(""), &atom!("class")).root() { None => {} Some(ref attr) => { - match attr.value().tokens() { + match attr.r().value().tokens() { None => {} Some(tokens) => { for token in tokens.iter() { @@ -1495,7 +1496,7 @@ impl<'a> ActivationElementHelpers<'a> for JSRef<'a, Element> { // https://html.spec.whatwg.org/multipage/interaction.html#nearest-activatable-element fn nearest_activable_element(self) -> Option<Temporary<Element>> { match self.as_maybe_activatable() { - Some(el) => Some(Temporary::from_rooted(*el.as_element().root())), + Some(el) => Some(Temporary::from_rooted(el.as_element().root().r())), None => { let node: JSRef<Node> = NodeCast::from_ref(self); node.ancestors() @@ -1526,7 +1527,7 @@ impl<'a> ActivationElementHelpers<'a> for JSRef<'a, Element> { // Step 4 let e = self.nearest_activable_element().root(); match e { - Some(el) => match el.as_maybe_activatable() { + Some(el) => match el.r().as_maybe_activatable() { Some(elem) => { // Step 5-6 elem.pre_click_activation(); diff --git a/components/script/dom/errorevent.rs b/components/script/dom/errorevent.rs index 9404fdf08b4..d70f5b3a9bc 100644 --- a/components/script/dom/errorevent.rs +++ b/components/script/dom/errorevent.rs @@ -64,15 +64,15 @@ impl ErrorEvent { colno: u32, error: JSVal) -> Temporary<ErrorEvent> { let ev = ErrorEvent::new_uninitialized(global).root(); - let event: JSRef<Event> = EventCast::from_ref(*ev); + let event: JSRef<Event> = EventCast::from_ref(ev.r()); event.InitEvent(type_, bubbles == EventBubbles::Bubbles, cancelable == EventCancelable::Cancelable); - *ev.message.borrow_mut() = message; - *ev.filename.borrow_mut() = filename; - ev.lineno.set(lineno); - ev.colno.set(colno); - ev.error.set(error); - Temporary::from_rooted(*ev) + *ev.r().message.borrow_mut() = message; + *ev.r().filename.borrow_mut() = filename; + ev.r().lineno.set(lineno); + ev.r().colno.set(colno); + ev.r().error.set(error); + Temporary::from_rooted(ev.r()) } pub fn Constructor(global: &GlobalRef, diff --git a/components/script/dom/event.rs b/components/script/dom/event.rs index cbbdada540d..0bfdb15336d 100644 --- a/components/script/dom/event.rs +++ b/components/script/dom/event.rs @@ -100,8 +100,8 @@ impl Event { bubbles: EventBubbles, cancelable: EventCancelable) -> Temporary<Event> { let event = Event::new_uninitialized(global).root(); - event.InitEvent(type_, bubbles == EventBubbles::Bubbles, cancelable == EventCancelable::Cancelable); - Temporary::from_rooted(*event) + event.r().InitEvent(type_, bubbles == EventBubbles::Bubbles, cancelable == EventCancelable::Cancelable); + Temporary::from_rooted(event.r()) } pub fn Constructor(global: &GlobalRef, diff --git a/components/script/dom/eventdispatcher.rs b/components/script/dom/eventdispatcher.rs index 412b0cdcb4d..d76c3002d8f 100644 --- a/components/script/dom/eventdispatcher.rs +++ b/components/script/dom/eventdispatcher.rs @@ -43,12 +43,12 @@ pub fn dispatch_event<'a, 'b>(target: JSRef<'a, EventTarget>, /* capturing */ for cur_target in chain.as_slice().iter().rev() { - let stopped = match cur_target.get_listeners_for(type_.as_slice(), ListenerPhase::Capturing) { + let stopped = match cur_target.r().get_listeners_for(type_.as_slice(), ListenerPhase::Capturing) { Some(listeners) => { - event.set_current_target(cur_target.deref().clone()); + event.set_current_target(cur_target.r()); for listener in listeners.iter() { // Explicitly drop any exception on the floor. - let _ = listener.HandleEvent_(**cur_target, event, ReportExceptions); + let _ = listener.HandleEvent_(cur_target.r(), event, ReportExceptions); if event.stop_immediate() { break; @@ -88,12 +88,12 @@ pub fn dispatch_event<'a, 'b>(target: JSRef<'a, EventTarget>, event.set_phase(EventPhase::Bubbling); for cur_target in chain.iter() { - let stopped = match cur_target.get_listeners_for(type_.as_slice(), ListenerPhase::Bubbling) { + let stopped = match cur_target.r().get_listeners_for(type_.as_slice(), ListenerPhase::Bubbling) { Some(listeners) => { - event.set_current_target(cur_target.deref().clone()); + event.set_current_target(cur_target.r()); for listener in listeners.iter() { // Explicitly drop any exception on the floor. - let _ = listener.HandleEvent_(**cur_target, event, ReportExceptions); + let _ = listener.HandleEvent_(cur_target.r(), event, ReportExceptions); if event.stop_immediate() { break; @@ -114,7 +114,7 @@ pub fn dispatch_event<'a, 'b>(target: JSRef<'a, EventTarget>, let target = event.GetTarget().root(); match target { Some(target) => { - let node: Option<JSRef<Node>> = NodeCast::to_ref(*target); + let node: Option<JSRef<Node>> = NodeCast::to_ref(target.r()); match node { Some(node) => { let vtable = vtable_for(&node); diff --git a/components/script/dom/htmlanchorelement.rs b/components/script/dom/htmlanchorelement.rs index a326c80af3b..f5c83f85783 100644 --- a/components/script/dom/htmlanchorelement.rs +++ b/components/script/dom/htmlanchorelement.rs @@ -62,11 +62,11 @@ impl<'a> PrivateHTMLAnchorElementHelpers for JSRef<'a, HTMLAnchorElement> { let attr = element.get_attribute(ns!(""), &atom!("href")).root(); match attr { Some(ref href) => { - let value = href.Value(); + let value = href.r().Value(); debug!("clicked on link to {:s}", value); let node: JSRef<Node> = NodeCast::from_ref(self); let doc = node.owner_doc().root(); - doc.load_anchor_href(value); + doc.r().load_anchor_href(value); } None => () } diff --git a/components/script/dom/htmlbodyelement.rs b/components/script/dom/htmlbodyelement.rs index 78a094dd016..8c03fde0a3e 100644 --- a/components/script/dom/htmlbodyelement.rs +++ b/components/script/dom/htmlbodyelement.rs @@ -56,12 +56,12 @@ impl HTMLBodyElement { impl<'a> HTMLBodyElementMethods for JSRef<'a, HTMLBodyElement> { fn GetOnunload(self) -> Option<EventHandlerNonNull> { let win = window_from_node(self).root(); - win.GetOnunload() + win.r().GetOnunload() } fn SetOnunload(self, listener: Option<EventHandlerNonNull>) { let win = window_from_node(self).root(); - win.SetOnunload(listener) + win.r().SetOnunload(listener) } } @@ -95,12 +95,12 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLBodyElement> { "onoffline", "ononline", "onpagehide", "onpageshow", "onpopstate", "onstorage", "onresize", "onunload", "onerror"]; let window = window_from_node(*self).root(); - let (cx, url, reflector) = (window.get_cx(), - window.get_url(), - window.reflector().get_jsobject()); + let (cx, url, reflector) = (window.r().get_cx(), + window.r().get_url(), + window.r().reflector().get_jsobject()); let evtarget: JSRef<EventTarget> = if FORWARDED_EVENTS.iter().any(|&event| name == event) { - EventTargetCast::from_ref(*window) + EventTargetCast::from_ref(window.r()) } else { EventTargetCast::from_ref(*self) }; diff --git a/components/script/dom/htmlbuttonelement.rs b/components/script/dom/htmlbuttonelement.rs index 5c60e79455d..ccbd5e27d00 100644 --- a/components/script/dom/htmlbuttonelement.rs +++ b/components/script/dom/htmlbuttonelement.rs @@ -49,7 +49,7 @@ impl HTMLButtonElement { impl<'a> HTMLButtonElementMethods for JSRef<'a, HTMLButtonElement> { fn Validity(self) -> Temporary<ValidityState> { let window = window_from_node(self).root(); - ValidityState::new(*window) + ValidityState::new(window.r()) } // http://www.whatwg.org/html/#dom-fe-disabled diff --git a/components/script/dom/htmlcanvaselement.rs b/components/script/dom/htmlcanvaselement.rs index 890e17ac452..32597f0fdb7 100644 --- a/components/script/dom/htmlcanvaselement.rs +++ b/components/script/dom/htmlcanvaselement.rs @@ -86,7 +86,7 @@ impl<'a> HTMLCanvasElementMethods for JSRef<'a, HTMLCanvasElement> { Some(self.context.or_init(|| { let window = window_from_node(self).root(); let (w, h) = (self.width.get() as i32, self.height.get() as i32); - CanvasRenderingContext2D::new(&GlobalRef::Window(*window), self, Size2D(w, h)) + CanvasRenderingContext2D::new(&GlobalRef::Window(window.r()), self, Size2D(w, h)) })) } } @@ -118,7 +118,7 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLCanvasElement> { if recreate { let (w, h) = (self.width.get() as i32, self.height.get() as i32); match self.context.get() { - Some(ref context) => context.root().recreate(Size2D(w, h)), + Some(ref context) => context.root().r().recreate(Size2D(w, h)), None => () } } @@ -146,7 +146,7 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLCanvasElement> { if recreate { let (w, h) = (self.width.get() as i32, self.height.get() as i32); match self.context.get() { - Some(ref context) => context.root().recreate(Size2D(w, h)), + Some(ref context) => context.root().r().recreate(Size2D(w, h)), None => () } } diff --git a/components/script/dom/htmlcollection.rs b/components/script/dom/htmlcollection.rs index af885c0b561..9cbff941c03 100644 --- a/components/script/dom/htmlcollection.rs +++ b/components/script/dom/htmlcollection.rs @@ -181,8 +181,8 @@ impl<'a> HTMLCollectionMethods for JSRef<'a, HTMLCollection> { CollectionTypeId::Static(ref elems) => elems.len() as u32, CollectionTypeId::Live(ref root, ref filter) => { let root = root.root(); - HTMLCollection::traverse(*root) - .filter(|element| filter.filter(*element, *root)) + HTMLCollection::traverse(root.r()) + .filter(|element| filter.filter(*element, root.r())) .count() as u32 } } @@ -197,8 +197,8 @@ impl<'a> HTMLCollectionMethods for JSRef<'a, HTMLCollection> { .map(|elem| Temporary::new(elem.clone())), CollectionTypeId::Live(ref root, ref filter) => { let root = root.root(); - HTMLCollection::traverse(*root) - .filter(|element| filter.filter(*element, *root)) + HTMLCollection::traverse(root.r()) + .filter(|element| filter.filter(*element, root.r())) .nth(index as uint) .clone() .map(Temporary::from_rooted) @@ -218,13 +218,13 @@ impl<'a> HTMLCollectionMethods for JSRef<'a, HTMLCollection> { CollectionTypeId::Static(ref elems) => elems.iter() .map(|elem| elem.root()) .find(|elem| { - elem.get_string_attribute(&atom!("name")) == key || - elem.get_string_attribute(&atom!("id")) == key }) - .map(|maybe_elem| Temporary::from_rooted(*maybe_elem)), + elem.r().get_string_attribute(&atom!("name")) == key || + elem.r().get_string_attribute(&atom!("id")) == key }) + .map(|maybe_elem| Temporary::from_rooted(maybe_elem.r())), CollectionTypeId::Live(ref root, ref filter) => { let root = root.root(); - HTMLCollection::traverse(*root) - .filter(|element| filter.filter(*element, *root)) + HTMLCollection::traverse(root.r()) + .filter(|element| filter.filter(*element, root.r())) .find(|elem| { elem.get_string_attribute(&atom!("name")) == key || elem.get_string_attribute(&atom!("id")) == key }) diff --git a/components/script/dom/htmldatalistelement.rs b/components/script/dom/htmldatalistelement.rs index 77c665dca7a..008845f80a1 100644 --- a/components/script/dom/htmldatalistelement.rs +++ b/components/script/dom/htmldatalistelement.rs @@ -52,7 +52,7 @@ impl<'a> HTMLDataListElementMethods for JSRef<'a, HTMLDataListElement> { let node: JSRef<Node> = NodeCast::from_ref(self); let filter = box HTMLDataListOptionsFilter; let window = window_from_node(node).root(); - HTMLCollection::create(*window, node, filter) + HTMLCollection::create(window.r(), node, filter) } } diff --git a/components/script/dom/htmlelement.rs b/components/script/dom/htmlelement.rs index 27cfc3712a3..687547ea02a 100644 --- a/components/script/dom/htmlelement.rs +++ b/components/script/dom/htmlelement.rs @@ -78,7 +78,7 @@ impl<'a> HTMLElementMethods for JSRef<'a, HTMLElement> { fn Style(self) -> Temporary<CSSStyleDeclaration> { self.style_decl.or_init(|| { let global = window_from_node(self).root(); - CSSStyleDeclaration::new(*global, self, CSSModificationAccess::ReadWrite) + CSSStyleDeclaration::new(global.r(), self, CSSModificationAccess::ReadWrite) }) } @@ -102,7 +102,7 @@ impl<'a> HTMLElementMethods for JSRef<'a, HTMLElement> { fn GetOnload(self) -> Option<EventHandlerNonNull> { if self.is_body_or_frameset() { let win = window_from_node(self).root(); - win.GetOnload() + win.r().GetOnload() } else { let target: JSRef<EventTarget> = EventTargetCast::from_ref(self); target.get_event_handler_common("load") @@ -112,7 +112,7 @@ impl<'a> HTMLElementMethods for JSRef<'a, HTMLElement> { fn SetOnload(self, listener: Option<EventHandlerNonNull>) { if self.is_body_or_frameset() { let win = window_from_node(self).root(); - win.SetOnload(listener) + win.r().SetOnload(listener) } else { let target: JSRef<EventTarget> = EventTargetCast::from_ref(self); target.set_event_handler_common("load", listener) @@ -167,7 +167,7 @@ impl<'a> HTMLElementCustomAttributeHelpers for JSRef<'a, HTMLElement> { let element: JSRef<Element> = ElementCast::from_ref(self); element.get_attribute(ns!(""), &Atom::from_slice(to_snake_case(name).as_slice())).map(|attr| { let attr = attr.root(); - attr.value().as_slice().into_string() + attr.r().value().as_slice().into_string() }) } @@ -192,9 +192,9 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLElement> { let name = attr.local_name().as_slice(); if name.starts_with("on") { let window = window_from_node(*self).root(); - let (cx, url, reflector) = (window.get_cx(), - window.get_url(), - window.reflector().get_jsobject()); + let (cx, url, reflector) = (window.r().get_cx(), + window.r().get_url(), + window.r().reflector().get_jsobject()); let evtarget: JSRef<EventTarget> = EventTargetCast::from_ref(*self); evtarget.set_event_handler_uncompiled(cx, url, reflector, name.slice_from(2), diff --git a/components/script/dom/htmlfieldsetelement.rs b/components/script/dom/htmlfieldsetelement.rs index d7cb3f1f7b6..3feedb27c63 100644 --- a/components/script/dom/htmlfieldsetelement.rs +++ b/components/script/dom/htmlfieldsetelement.rs @@ -61,12 +61,12 @@ impl<'a> HTMLFieldSetElementMethods for JSRef<'a, HTMLFieldSetElement> { let node: JSRef<Node> = NodeCast::from_ref(self); let filter = box ElementsFilter; let window = window_from_node(node).root(); - HTMLCollection::create(*window, node, filter) + HTMLCollection::create(window.r(), node, filter) } fn Validity(self) -> Temporary<ValidityState> { let window = window_from_node(self).root(); - ValidityState::new(*window) + ValidityState::new(window.r()) } // http://www.whatwg.org/html/#dom-fieldset-disabled diff --git a/components/script/dom/htmlformelement.rs b/components/script/dom/htmlformelement.rs index e58aaed0650..e313ac05195 100644 --- a/components/script/dom/htmlformelement.rs +++ b/components/script/dom/htmlformelement.rs @@ -152,17 +152,17 @@ impl<'a> HTMLFormElementHelpers for JSRef<'a, HTMLFormElement> { // Step 1 let doc = document_from_node(self).root(); let win = window_from_node(self).root(); - let base = doc.url(); + let base = doc.r().url(); // TODO: Handle browsing contexts // TODO: Handle validation - let event = Event::new(GlobalRef::Window(*win), + let event = Event::new(GlobalRef::Window(win.r()), "submit".into_string(), EventBubbles::Bubbles, EventCancelable::Cancelable).root(); - event.set_trusted(true); + event.r().set_trusted(true); let target: JSRef<EventTarget> = EventTargetCast::from_ref(self); - target.DispatchEvent(*event).ok(); - if event.DefaultPrevented() { + target.DispatchEvent(event.r()).ok(); + if event.r().DefaultPrevented() { return; } // Step 6 @@ -204,7 +204,7 @@ impl<'a> HTMLFormElementHelpers for JSRef<'a, HTMLFormElement> { } // This is wrong. https://html.spec.whatwg.org/multipage/forms.html#planned-navigation - win.script_chan().send(ScriptMsg::TriggerLoad(win.page().id, load_data)); + win.r().script_chan().send(ScriptMsg::TriggerLoad(win.r().page().id, load_data)); } fn get_form_dataset<'b>(self, submitter: Option<FormSubmitter<'b>>) -> Vec<FormDatum> { @@ -342,13 +342,13 @@ impl<'a> HTMLFormElementHelpers for JSRef<'a, HTMLFormElement> { } let win = window_from_node(self).root(); - let event = Event::new(GlobalRef::Window(*win), + let event = Event::new(GlobalRef::Window(win.r()), "reset".into_string(), EventBubbles::Bubbles, EventCancelable::Cancelable).root(); let target: JSRef<EventTarget> = EventTargetCast::from_ref(self); - target.DispatchEvent(*event).ok(); - if event.DefaultPrevented() { + target.DispatchEvent(event.r()).ok(); + if event.r().DefaultPrevented() { return; } @@ -484,10 +484,10 @@ pub trait FormControl<'a> : Copy { let owner = elem.get_string_attribute(&atom!("form")); if !owner.is_empty() { let doc = document_from_node(elem).root(); - let owner = doc.GetElementById(owner).root(); + let owner = doc.r().GetElementById(owner).root(); match owner { Some(o) => { - let maybe_form: Option<JSRef<HTMLFormElement>> = HTMLFormElementCast::to_ref(*o); + let maybe_form: Option<JSRef<HTMLFormElement>> = HTMLFormElementCast::to_ref(o.r()); if maybe_form.is_some() { return maybe_form.map(Temporary::from_rooted); } @@ -507,7 +507,7 @@ pub trait FormControl<'a> : Copy { if self.to_element().has_attribute(attr) { input(self) } else { - self.form_owner().map_or("".into_string(), |t| owner(*t.root())) + self.form_owner().map_or("".into_string(), |t| owner(t.root().r())) } } fn to_element(self) -> JSRef<'a, Element>; diff --git a/components/script/dom/htmliframeelement.rs b/components/script/dom/htmliframeelement.rs index 11da7996058..b4979b488c1 100644 --- a/components/script/dom/htmliframeelement.rs +++ b/components/script/dom/htmliframeelement.rs @@ -87,12 +87,12 @@ impl<'a> HTMLIFrameElementHelpers for JSRef<'a, HTMLIFrameElement> { fn get_url(self) -> Option<Url> { let element: JSRef<Element> = ElementCast::from_ref(self); element.get_attribute(ns!(""), &atom!("src")).root().and_then(|src| { - let url = src.value(); + let url = src.r().value(); if url.as_slice().is_empty() { None } else { let window = window_from_node(self).root(); - UrlParser::new().base_url(&window.page().get_url()) + UrlParser::new().base_url(&window.r().page().get_url()) .parse(url.as_slice()).ok() } }) @@ -188,10 +188,10 @@ impl<'a> HTMLIFrameElementMethods for JSRef<'a, HTMLIFrameElement> { Some(self_url) => self_url, None => return None, }; - let win_url = window_from_node(self).root().page().get_url(); + let win_url = window_from_node(self).root().r().page().get_url(); if UrlHelper::SameOrigin(&self_url, &win_url) { - Some(window.Document()) + Some(window.r().Document()) } else { None } diff --git a/components/script/dom/htmlimageelement.rs b/components/script/dom/htmlimageelement.rs index e52d902db5e..07731c98616 100644 --- a/components/script/dom/htmlimageelement.rs +++ b/components/script/dom/htmlimageelement.rs @@ -45,7 +45,7 @@ impl<'a> PrivateHTMLImageElementHelpers for JSRef<'a, HTMLImageElement> { fn update_image(self, value: Option<(DOMString, &Url)>) { let node: JSRef<Node> = NodeCast::from_ref(self); let document = node.owner_doc().root(); - let window = document.window().root(); + let window = document.r().window().root(); let image_cache = window.image_cache_task(); match value { None => { @@ -186,7 +186,7 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLImageElement> { match attr.local_name() { &atom!("src") => { let window = window_from_node(*self).root(); - let url = window.get_url(); + let url = window.r().get_url(); self.update_image(Some((attr.value().as_slice().into_string(), &url))); }, _ => () diff --git a/components/script/dom/htmlinputelement.rs b/components/script/dom/htmlinputelement.rs index ecf1ecf9d23..df34dcd8e39 100644 --- a/components/script/dom/htmlinputelement.rs +++ b/components/script/dom/htmlinputelement.rs @@ -307,7 +307,7 @@ fn broadcast_radio_checked(broadcaster: JSRef<HTMLInputElement>, group: Option<& //TODO: if not in document, use root ancestor instead of document let owner = broadcaster.form_owner().root(); let doc = document_from_node(broadcaster).root(); - let doc_node: JSRef<Node> = NodeCast::from_ref(*doc); + let doc_node: JSRef<Node> = NodeCast::from_ref(doc.r()); // There is no DOM tree manipulation here, so this is safe let mut iter = unsafe { @@ -342,7 +342,7 @@ impl<'a> HTMLInputElementHelpers for JSRef<'a, HTMLInputElement> { fn force_relayout(self) { let doc = document_from_node(self).root(); let node: JSRef<Node> = NodeCast::from_ref(self); - doc.content_changed(node, NodeDamage::OtherNodeDamage) + doc.r().content_changed(node, NodeDamage::OtherNodeDamage) } fn radio_group_updated(self, group: Option<&str>) { @@ -356,7 +356,7 @@ impl<'a> HTMLInputElementHelpers for JSRef<'a, HTMLInputElement> { let elem: JSRef<Element> = ElementCast::from_ref(self); elem.get_attribute(ns!(""), &atom!("name")) .root() - .map(|name| name.Value()) + .map(|name| name.r().Value()) } fn update_checked_state(self, checked: bool, dirty: bool) { @@ -550,7 +550,7 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLInputElement> { //TODO: set the editing position for text inputs let doc = document_from_node(*self).root(); - doc.request_focus(ElementCast::from_ref(*self)); + doc.r().request_focus(ElementCast::from_ref(*self)); } else if "keydown" == event.Type().as_slice() && !event.DefaultPrevented() && (self.input_type.get() == InputType::InputText || self.input_type.get() == InputType::InputPassword) { @@ -631,7 +631,7 @@ impl<'a> Activatable for JSRef<'a, HTMLInputElement> { //TODO: if not in document, use root ancestor instead of document let owner = self.form_owner().root(); let doc = document_from_node(*self).root(); - let doc_node: JSRef<Node> = NodeCast::from_ref(*doc); + let doc_node: JSRef<Node> = NodeCast::from_ref(doc.r()); let group = self.get_radio_group_name();; // Safe since we only manipulate the DOM tree after finding an element @@ -684,11 +684,11 @@ impl<'a> Activatable for JSRef<'a, HTMLInputElement> { Some(o) => { // Avoiding iterating through the whole tree here, instead // we can check if the conditions for radio group siblings apply - if name == o.get_radio_group_name() && // TODO should be compatibility caseless - self.form_owner() == o.form_owner() && + if name == o.r().get_radio_group_name() && // TODO should be compatibility caseless + self.form_owner() == o.r().form_owner() && // TODO Both a and b are in the same home subtree - o.input_type.get() == InputType::InputRadio { - o.SetChecked(true); + o.r().input_type.get() == InputType::InputRadio { + o.r().SetChecked(true); } else { self.SetChecked(false); } @@ -716,8 +716,8 @@ impl<'a> Activatable for JSRef<'a, HTMLInputElement> { // FIXME (Manishearth): support document owners (needs ability to get parent browsing context) if self.mutable() /* and document owner is fully active */ { self.form_owner().map(|o| { - o.root().submit(SubmittedFrom::NotFromFormSubmitMethod, - FormSubmitter::InputElement(self.clone())) + o.root().r().submit(SubmittedFrom::NotFromFormSubmitMethod, + FormSubmitter::InputElement(self.clone())) }); } }, @@ -726,7 +726,7 @@ impl<'a> Activatable for JSRef<'a, HTMLInputElement> { // FIXME (Manishearth): support document owners (needs ability to get parent browsing context) if self.mutable() /* and document owner is fully active */ { self.form_owner().map(|o| { - o.root().reset(ResetFrom::NotFromFormResetMethod) + o.root().r().reset(ResetFrom::NotFromFormResetMethod) }); } }, @@ -735,21 +735,21 @@ impl<'a> Activatable for JSRef<'a, HTMLInputElement> { // https://html.spec.whatwg.org/multipage/forms.html#radio-button-state-(type=radio):activation-behavior if self.mutable() { let win = window_from_node(*self).root(); - let event = Event::new(GlobalRef::Window(*win), + let event = Event::new(GlobalRef::Window(win.r()), "input".into_string(), EventBubbles::Bubbles, EventCancelable::NotCancelable).root(); - event.set_trusted(true); + event.r().set_trusted(true); let target: JSRef<EventTarget> = EventTargetCast::from_ref(*self); - target.DispatchEvent(*event).ok(); + target.DispatchEvent(event.r()).ok(); - let event = Event::new(GlobalRef::Window(*win), + let event = Event::new(GlobalRef::Window(win.r()), "change".into_string(), EventBubbles::Bubbles, EventCancelable::NotCancelable).root(); - event.set_trusted(true); + event.r().set_trusted(true); let target: JSRef<EventTarget> = EventTargetCast::from_ref(*self); - target.DispatchEvent(*event).ok(); + target.DispatchEvent(event.r()).ok(); } }, _ => () @@ -759,7 +759,7 @@ impl<'a> Activatable for JSRef<'a, HTMLInputElement> { // https://html.spec.whatwg.org/multipage/forms.html#implicit-submission fn implicit_submission(&self, ctrlKey: bool, shiftKey: bool, altKey: bool, metaKey: bool) { let doc = document_from_node(*self).root(); - let node: JSRef<Node> = NodeCast::from_ref(*doc); + let node: JSRef<Node> = NodeCast::from_ref(doc.r()); let owner = self.form_owner(); if owner.is_none() || ElementCast::from_ref(*self).click_in_progress() { return; diff --git a/components/script/dom/htmllinkelement.rs b/components/script/dom/htmllinkelement.rs index 3a78e4a0c02..2e813d5e501 100644 --- a/components/script/dom/htmllinkelement.rs +++ b/components/script/dom/htmllinkelement.rs @@ -53,7 +53,7 @@ impl HTMLLinkElement { fn get_attr(element: JSRef<Element>, name: &Atom) -> Option<String> { let elem = element.get_attribute(ns!(""), name).root(); - elem.map(|e| e.value().as_slice().into_string()) + elem.map(|e| e.r().value().as_slice().into_string()) } fn is_stylesheet(value: &Option<String>) -> bool { @@ -127,7 +127,7 @@ trait PrivateHTMLLinkElementHelpers { impl<'a> PrivateHTMLLinkElementHelpers for JSRef<'a, HTMLLinkElement> { fn handle_stylesheet_url(self, href: &str) { let window = window_from_node(self).root(); - match UrlParser::new().base_url(&window.page().get_url()).parse(href) { + match UrlParser::new().base_url(&window.r().page().get_url()).parse(href) { Ok(url) => { let LayoutChan(ref layout_chan) = window.page().layout_chan; layout_chan.send(Msg::LoadStylesheet(url)); diff --git a/components/script/dom/htmlobjectelement.rs b/components/script/dom/htmlobjectelement.rs index 81b0e31371b..5dda50c3d7c 100644 --- a/components/script/dom/htmlobjectelement.rs +++ b/components/script/dom/htmlobjectelement.rs @@ -62,8 +62,8 @@ impl<'a> ProcessDataURL for JSRef<'a, HTMLObjectElement> { let elem: JSRef<Element> = ElementCast::from_ref(*self); // TODO: support other values - match (elem.get_attribute(ns!(""), &atom!("type")).map(|x| x.root().Value()), - elem.get_attribute(ns!(""), &atom!("data")).map(|x| x.root().Value())) { + match (elem.get_attribute(ns!(""), &atom!("type")).map(|x| x.root().r().Value()), + elem.get_attribute(ns!(""), &atom!("data")).map(|x| x.root().r().Value())) { (None, Some(uri)) => { if is_image_data(uri.as_slice()) { let data_url = Url::parse(uri.as_slice()).unwrap(); @@ -84,7 +84,7 @@ pub fn is_image_data(uri: &str) -> bool { impl<'a> HTMLObjectElementMethods for JSRef<'a, HTMLObjectElement> { fn Validity(self) -> Temporary<ValidityState> { let window = window_from_node(self).root(); - ValidityState::new(*window) + ValidityState::new(window.r()) } // https://html.spec.whatwg.org/multipage/embedded-content.html#dom-object-type @@ -109,7 +109,7 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLObjectElement> { match attr.local_name() { &atom!("data") => { let window = window_from_node(*self).root(); - self.process_data_url(window.image_cache_task().clone()); + self.process_data_url(window.r().image_cache_task().clone()); }, _ => () } diff --git a/components/script/dom/htmloutputelement.rs b/components/script/dom/htmloutputelement.rs index 5d7b8f6e198..d453dd78ab3 100644 --- a/components/script/dom/htmloutputelement.rs +++ b/components/script/dom/htmloutputelement.rs @@ -42,7 +42,7 @@ impl HTMLOutputElement { impl<'a> HTMLOutputElementMethods for JSRef<'a, HTMLOutputElement> { fn Validity(self) -> Temporary<ValidityState> { let window = window_from_node(self).root(); - ValidityState::new(*window) + ValidityState::new(window.r()) } } diff --git a/components/script/dom/htmlscriptelement.rs b/components/script/dom/htmlscriptelement.rs index eb1dfbabbd7..19c1e1f7a0c 100644 --- a/components/script/dom/htmlscriptelement.rs +++ b/components/script/dom/htmlscriptelement.rs @@ -175,11 +175,11 @@ impl<'a> HTMLScriptElementHelpers for JSRef<'a, HTMLScriptElement> { let (source, url) = match element.get_attribute(ns!(""), &atom!("src")).root() { Some(src) => { - if src.deref().Value().is_empty() { + if src.r().Value().is_empty() { // TODO: queue a task to fire a simple event named `error` at the element return; } - match UrlParser::new().base_url(&base_url).parse(src.deref().Value().as_slice()) { + match UrlParser::new().base_url(&base_url).parse(src.r().Value().as_slice()) { Ok(url) => { // TODO: Do a potentially CORS-enabled fetch with the mode being the current // state of the element's `crossorigin` content attribute, the origin being @@ -192,14 +192,14 @@ impl<'a> HTMLScriptElementHelpers for JSRef<'a, HTMLScriptElement> { (source, metadata.final_url) } Err(_) => { - error!("error loading script {}", src.deref().Value()); + error!("error loading script {}", src.r().Value()); return; } } } Err(_) => { // TODO: queue a task to fire a simple event named `error` at the element - error!("error parsing URL for script {}", src.deref().Value()); + error!("error parsing URL for script {}", src.r().Value()); return; } } @@ -207,20 +207,20 @@ impl<'a> HTMLScriptElementHelpers for JSRef<'a, HTMLScriptElement> { None => (text, base_url) }; - window.evaluate_script_with_result(source.as_slice(), url.serialize().as_slice()); + window.r().evaluate_script_with_result(source.as_slice(), url.serialize().as_slice()); - let event = Event::new(GlobalRef::Window(*window), + let event = Event::new(GlobalRef::Window(window.r()), "load".into_string(), EventBubbles::DoesNotBubble, EventCancelable::NotCancelable).root(); - event.set_trusted(true); + event.r().set_trusted(true); let target: JSRef<EventTarget> = EventTargetCast::from_ref(self); - target.dispatch_event(*event); + target.dispatch_event(event.r()); } fn is_javascript(self) -> bool { let element: JSRef<Element> = ElementCast::from_ref(self); - match element.get_attribute(ns!(""), &atom!("type")).root().map(|s| s.Value()) { + match element.get_attribute(ns!(""), &atom!("type")).root().map(|s| s.r().Value()) { Some(ref s) if s.is_empty() => { // type attr exists, but empty means js debug!("script type empty, inferring js"); @@ -234,7 +234,7 @@ impl<'a> HTMLScriptElementHelpers for JSRef<'a, HTMLScriptElement> { debug!("no script type"); match element.get_attribute(ns!(""), &atom!("language")) .root() - .map(|s| s.Value()) { + .map(|s| s.r().Value()) { Some(ref s) if s.is_empty() => { debug!("script language empty, inferring js"); true diff --git a/components/script/dom/htmlselectelement.rs b/components/script/dom/htmlselectelement.rs index 2ba3786c6cf..7daa654a505 100644 --- a/components/script/dom/htmlselectelement.rs +++ b/components/script/dom/htmlselectelement.rs @@ -50,7 +50,7 @@ impl HTMLSelectElement { impl<'a> HTMLSelectElementMethods for JSRef<'a, HTMLSelectElement> { fn Validity(self) -> Temporary<ValidityState> { let window = window_from_node(self).root(); - ValidityState::new(*window) + ValidityState::new(window.r()) } // Note: this function currently only exists for test_union.html. diff --git a/components/script/dom/htmlserializer.rs b/components/script/dom/htmlserializer.rs index 97526ce5f4e..0ff300f97f1 100644 --- a/components/script/dom/htmlserializer.rs +++ b/components/script/dom/htmlserializer.rs @@ -71,8 +71,8 @@ fn serialize_comment(comment: JSRef<Comment>, html: &mut String) { fn serialize_text(text: JSRef<Text>, html: &mut String) { let text_node: JSRef<Node> = NodeCast::from_ref(text); match text_node.parent_node().map(|node| node.root()) { - Some(ref parent) if parent.is_element() => { - let elem: JSRef<Element> = ElementCast::to_ref(**parent).unwrap(); + Some(ref parent) if parent.r().is_element() => { + let elem: JSRef<Element> = ElementCast::to_ref(parent.r()).unwrap(); match elem.local_name().as_slice() { "style" | "script" | "xmp" | "iframe" | "noembed" | "noframes" | "plaintext" | @@ -105,7 +105,7 @@ fn serialize_elem(elem: JSRef<Element>, open_elements: &mut Vec<String>, html: & html.push_str(elem.local_name().as_slice()); for attr in elem.attrs().iter() { let attr = attr.root(); - serialize_attr(*attr, html); + serialize_attr(attr.r(), html); }; html.push('>'); @@ -113,8 +113,8 @@ fn serialize_elem(elem: JSRef<Element>, open_elements: &mut Vec<String>, html: & "pre" | "listing" | "textarea" if *elem.namespace() == ns!(HTML) => { let node: JSRef<Node> = NodeCast::from_ref(elem); match node.first_child().map(|child| child.root()) { - Some(ref child) if child.is_text() => { - let text: JSRef<CharacterData> = CharacterDataCast::to_ref(**child).unwrap(); + Some(ref child) if child.r().is_text() => { + let text: JSRef<CharacterData> = CharacterDataCast::to_ref(child.r()).unwrap(); if text.data().len() > 0 && text.data().as_slice().char_at(0) == '\n' { html.push('\x0A'); } diff --git a/components/script/dom/htmltableelement.rs b/components/script/dom/htmltableelement.rs index b21950b096c..8dbb3e7685a 100644 --- a/components/script/dom/htmltableelement.rs +++ b/components/script/dom/htmltableelement.rs @@ -75,7 +75,7 @@ impl<'a> HTMLTableElementMethods for JSRef<'a, HTMLTableElement> { match old_caption { Some(htmlelem) => { let htmlelem_root = htmlelem.root(); - let old_caption_node: JSRef<Node> = NodeCast::from_ref(*htmlelem_root); + let old_caption_node: JSRef<Node> = NodeCast::from_ref(htmlelem_root.r()); assert!(node.RemoveChild(old_caption_node).is_ok()); } None => () diff --git a/components/script/dom/htmltextareaelement.rs b/components/script/dom/htmltextareaelement.rs index 88fd31fdeaa..28e79ab7686 100644 --- a/components/script/dom/htmltextareaelement.rs +++ b/components/script/dom/htmltextareaelement.rs @@ -193,7 +193,7 @@ impl<'a> PrivateHTMLTextAreaElementHelpers for JSRef<'a, HTMLTextAreaElement> { fn force_relayout(self) { let doc = document_from_node(self).root(); let node: JSRef<Node> = NodeCast::from_ref(self); - doc.content_changed(node, NodeDamage::OtherNodeDamage) + doc.r().content_changed(node, NodeDamage::OtherNodeDamage) } } @@ -312,7 +312,7 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLTextAreaElement> { //TODO: set the editing position for text inputs let doc = document_from_node(*self).root(); - doc.request_focus(ElementCast::from_ref(*self)); + doc.r().request_focus(ElementCast::from_ref(*self)); } else if "keydown" == event.Type().as_slice() && !event.DefaultPrevented() { let keyevent: Option<JSRef<KeyboardEvent>> = KeyboardEventCast::to_ref(event); keyevent.map(|event| { diff --git a/components/script/dom/htmltitleelement.rs b/components/script/dom/htmltitleelement.rs index c7f42c85eb9..8f750b2f3fc 100644 --- a/components/script/dom/htmltitleelement.rs +++ b/components/script/dom/htmltitleelement.rs @@ -74,7 +74,7 @@ impl<'a> VirtualMethods for JSRef<'a, HTMLTitleElement> { let node: JSRef<Node> = NodeCast::from_ref(*self); if is_in_doc { let document = node.owner_doc().root(); - document.send_title_to_compositor() + document.r().send_title_to_compositor() } } } diff --git a/components/script/dom/keyboardevent.rs b/components/script/dom/keyboardevent.rs index e409a20b5e0..c7dfde02aac 100644 --- a/components/script/dom/keyboardevent.rs +++ b/components/script/dom/keyboardevent.rs @@ -82,17 +82,17 @@ impl KeyboardEvent { char_code: Option<u32>, key_code: u32) -> Temporary<KeyboardEvent> { let ev = KeyboardEvent::new_uninitialized(window).root(); - ev.deref().InitKeyboardEvent(type_, canBubble, cancelable, view, key, location, - "".into_string(), repeat, "".into_string()); - *ev.code.borrow_mut() = code; - ev.ctrl.set(ctrlKey); - ev.alt.set(altKey); - ev.shift.set(shiftKey); - ev.meta.set(metaKey); - ev.char_code.set(char_code); - ev.key_code.set(key_code); - ev.is_composing.set(isComposing); - Temporary::from_rooted(*ev) + ev.r().InitKeyboardEvent(type_, canBubble, cancelable, view, key, location, + "".into_string(), repeat, "".into_string()); + *ev.r().code.borrow_mut() = code; + ev.r().ctrl.set(ctrlKey); + ev.r().alt.set(altKey); + ev.r().shift.set(shiftKey); + ev.r().meta.set(metaKey); + ev.r().char_code.set(char_code); + ev.r().key_code.set(key_code); + ev.r().is_composing.set(isComposing); + Temporary::from_rooted(ev.r()) } pub fn Constructor(global: &GlobalRef, diff --git a/components/script/dom/macros.rs b/components/script/dom/macros.rs index 066c967c594..e67f04b30ba 100644 --- a/components/script/dom/macros.rs +++ b/components/script/dom/macros.rs @@ -87,7 +87,7 @@ macro_rules! make_url_or_base_getter( match url.as_slice() { "" => { let window = window_from_node(self).root(); - window.get_url().serialize() + window.r().get_url().serialize() }, _ => url } diff --git a/components/script/dom/messageevent.rs b/components/script/dom/messageevent.rs index 369b3d1ebf7..5b6489b41af 100644 --- a/components/script/dom/messageevent.rs +++ b/components/script/dom/messageevent.rs @@ -58,9 +58,9 @@ impl MessageEvent { data: JSVal, origin: DOMString, lastEventId: DOMString) -> Temporary<MessageEvent> { let ev = MessageEvent::new_initialized(global, data, origin, lastEventId).root(); - let event: JSRef<Event> = EventCast::from_ref(*ev); + let event: JSRef<Event> = EventCast::from_ref(ev.r()); event.InitEvent(type_, bubbles, cancelable); - Temporary::from_rooted(*ev) + Temporary::from_rooted(ev.r()) } pub fn Constructor(global: &GlobalRef, @@ -80,7 +80,7 @@ impl MessageEvent { let messageevent = MessageEvent::new( scope, "message".into_string(), false, false, message, "".into_string(), "".into_string()).root(); - let event: JSRef<Event> = EventCast::from_ref(*messageevent); + let event: JSRef<Event> = EventCast::from_ref(messageevent.r()); target.dispatch_event(event); } } diff --git a/components/script/dom/mouseevent.rs b/components/script/dom/mouseevent.rs index 38fe5119ea9..086137a25af 100644 --- a/components/script/dom/mouseevent.rs +++ b/components/script/dom/mouseevent.rs @@ -79,11 +79,11 @@ impl MouseEvent { button: i16, relatedTarget: Option<JSRef<EventTarget>>) -> Temporary<MouseEvent> { let ev = MouseEvent::new_uninitialized(window).root(); - ev.InitMouseEvent(type_, canBubble, cancelable, view, detail, - screenX, screenY, clientX, clientY, - ctrlKey, altKey, shiftKey, metaKey, - button, relatedTarget); - Temporary::from_rooted(*ev) + ev.r().InitMouseEvent(type_, canBubble, cancelable, view, detail, + screenX, screenY, clientX, clientY, + ctrlKey, altKey, shiftKey, metaKey, + button, relatedTarget); + Temporary::from_rooted(ev.r()) } pub fn Constructor(global: &GlobalRef, diff --git a/components/script/dom/namednodemap.rs b/components/script/dom/namednodemap.rs index e0bc06b980d..74989ec3ddc 100644 --- a/components/script/dom/namednodemap.rs +++ b/components/script/dom/namednodemap.rs @@ -33,11 +33,11 @@ impl NamedNodeMap { impl<'a> NamedNodeMapMethods for JSRef<'a, NamedNodeMap> { fn Length(self) -> u32 { - self.owner.root().attrs().len() as u32 + self.owner.root().r().attrs().len() as u32 } fn Item(self, index: u32) -> Option<Temporary<Attr>> { - self.owner.root().attrs().as_slice().get(index as uint).map(|x| Temporary::new(x.clone())) + self.owner.root().r().attrs().as_slice().get(index as uint).map(|x| Temporary::new(x.clone())) } fn IndexedGetter(self, index: u32, found: &mut bool) -> Option<Temporary<Attr>> { diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index 595410a4818..45bcc2a8ce6 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -282,8 +282,8 @@ impl<'a> PrivateNodeHelpers for JSRef<'a, Node> { } let parent = self.parent_node().root(); - parent.map(|parent| vtable_for(&*parent).child_inserted(self)); - document.content_and_heritage_changed(self, NodeDamage::OtherNodeDamage); + parent.map(|parent| vtable_for(&parent.r()).child_inserted(self)); + document.r().content_and_heritage_changed(self, NodeDamage::OtherNodeDamage); } // http://dom.spec.whatwg.org/#node-is-removed @@ -314,8 +314,8 @@ impl<'a> PrivateNodeHelpers for JSRef<'a, Node> { self.first_child.assign(Some(new_child)); }, Some(prev_sibling) => { - prev_sibling.next_sibling.assign(Some(new_child)); - new_child.prev_sibling.assign(Some(*prev_sibling)); + prev_sibling.r().next_sibling.assign(Some(new_child)); + new_child.prev_sibling.assign(Some(prev_sibling.r())); }, } before.prev_sibling.assign(Some(new_child)); @@ -325,9 +325,9 @@ impl<'a> PrivateNodeHelpers for JSRef<'a, Node> { match self.last_child().root() { None => self.first_child.assign(Some(new_child)), Some(last_child) => { - assert!(last_child.next_sibling().is_none()); - last_child.next_sibling.assign(Some(new_child)); - new_child.prev_sibling.assign(Some(*last_child)); + assert!(last_child.r().next_sibling().is_none()); + last_child.r().next_sibling.assign(Some(new_child)); + new_child.prev_sibling.assign(Some(last_child.r())); } } @@ -349,7 +349,7 @@ impl<'a> PrivateNodeHelpers for JSRef<'a, Node> { self.first_child.assign(child.next_sibling.get()); } Some(prev_sibling) => { - prev_sibling.next_sibling.assign(child.next_sibling.get()); + prev_sibling.r().next_sibling.assign(child.next_sibling.get()); } } @@ -358,7 +358,7 @@ impl<'a> PrivateNodeHelpers for JSRef<'a, Node> { self.last_child.assign(child.prev_sibling.get()); } Some(next_sibling) => { - next_sibling.prev_sibling.assign(child.prev_sibling.get()); + next_sibling.r().prev_sibling.assign(child.prev_sibling.get()); } } @@ -682,7 +682,7 @@ impl<'a> NodeHelpers<'a> for JSRef<'a, Node> { Some(parent) => parent, }; - for sibling in parent.root().children() { + for sibling in parent.root().r().children() { sibling.set_has_dirty_siblings(true); } @@ -726,11 +726,11 @@ impl<'a> NodeHelpers<'a> for JSRef<'a, Node> { } fn get_bounding_content_box(self) -> Rect<Au> { - window_from_node(self).root().page().content_box_query(self.to_trusted_node_address()) + window_from_node(self).root().r().page().content_box_query(self.to_trusted_node_address()) } fn get_content_boxes(self) -> Vec<Rect<Au>> { - window_from_node(self).root().page().content_boxes_query(self.to_trusted_node_address()) + window_from_node(self).root().r().page().content_boxes_query(self.to_trusted_node_address()) } // http://dom.spec.whatwg.org/#dom-parentnode-queryselector @@ -781,7 +781,7 @@ impl<'a> NodeHelpers<'a> for JSRef<'a, Node> { unsafe { self.query_selector_iter(selectors).map(|mut iter| { let window = window_from_node(self).root(); - NodeList::new_simple_list(*window, iter.collect()) + NodeList::new_simple_list(window.r(), iter.collect()) }) } } @@ -802,7 +802,7 @@ impl<'a> NodeHelpers<'a> for JSRef<'a, Node> { } fn is_in_html_doc(self) -> bool { - self.owner_doc().root().is_html_document() + self.owner_doc().root().r().is_html_document() } fn children(self) -> NodeChildrenIterator<'a> { @@ -825,7 +825,7 @@ impl<'a> NodeHelpers<'a> for JSRef<'a, Node> { fn remove_self(self) { match self.parent_node().root() { - Some(parent) => parent.remove_child(self), + Some(parent) => parent.r().remove_child(self), None => () } } @@ -843,11 +843,11 @@ impl<'a> NodeHelpers<'a> for JSRef<'a, Node> { NodeInfo { uniqueId: self.unique_id.borrow().clone(), baseURI: self.GetBaseURI().unwrap_or("".into_string()), - parent: self.GetParentNode().root().map(|node| node.get_unique_id()).unwrap_or("".into_string()), + parent: self.GetParentNode().root().map(|node| node.r().get_unique_id()).unwrap_or("".into_string()), nodeType: self.NodeType() as uint, namespaceURI: "".into_string(), //FIXME nodeName: self.NodeName(), - numChildren: self.ChildNodes().root().Length() as uint, + numChildren: self.ChildNodes().root().r().Length() as uint, //FIXME doctype nodes only name: "".into_string(), @@ -861,8 +861,9 @@ impl<'a> NodeHelpers<'a> for JSRef<'a, Node> { isDocumentElement: self.owner_doc().root() + .r() .GetDocumentElement() - .map(|elem| NodeCast::from_ref(*elem.root()) == self) + .map(|elem| NodeCast::from_ref(elem.root().r()) == self) .unwrap_or(false), shortValue: self.GetNodeValue().unwrap_or("".into_string()), //FIXME: truncate @@ -1159,7 +1160,7 @@ impl Node { wrap_fn: extern "Rust" fn(*mut JSContext, GlobalRef, Box<N>) -> Temporary<N>) -> Temporary<N> { let window = document.window().root(); - reflect_dom_object(node, GlobalRef::Window(*window), wrap_fn) + reflect_dom_object(node, GlobalRef::Window(window.r()), wrap_fn) } pub fn new_inherited(type_id: NodeTypeId, doc: JSRef<Document>) -> Node { @@ -1210,14 +1211,14 @@ impl Node { // Step 1. match node.parent_node().root() { Some(parent) => { - Node::remove(node, *parent, SuppressObserver::Unsuppressed); + Node::remove(node, parent.r(), SuppressObserver::Unsuppressed); } None => (), } // Step 2. let node_doc = document_from_node(node).root(); - if *node_doc != document { + if node_doc.r() != document { for descendant in node.traverse_preorder() { descendant.set_owner_doc(document); } @@ -1429,7 +1430,7 @@ impl Node { match node { Some(node) => { let document = document_from_node(parent).root(); - Node::adopt(node, *document); + Node::adopt(node, document.r()); } None => (), } @@ -1519,16 +1520,16 @@ impl Node { let doctype: JSRef<DocumentType> = DocumentTypeCast::to_ref(node).unwrap(); let doctype = DocumentType::new(doctype.name().clone(), Some(doctype.public_id().clone()), - Some(doctype.system_id().clone()), *document); + Some(doctype.system_id().clone()), document.r()); NodeCast::from_temporary(doctype) }, NodeTypeId::DocumentFragment => { - let doc_fragment = DocumentFragment::new(*document); + let doc_fragment = DocumentFragment::new(document.r()); NodeCast::from_temporary(doc_fragment) }, NodeTypeId::Comment => { let comment: JSRef<Comment> = CommentCast::to_ref(node).unwrap(); - let comment = Comment::new(comment.characterdata().data().clone(), *document); + let comment = Comment::new(comment.characterdata().data().clone(), document.r()); NodeCast::from_temporary(comment) }, NodeTypeId::Document => { @@ -1538,7 +1539,7 @@ impl Node { false => IsHTMLDocument::NonHTMLDocument, }; let window = document.window().root(); - let document = Document::new(*window, Some(document.url().clone()), + let document = Document::new(window.r(), Some(document.url().clone()), is_html_doc, None, DocumentSource::NotFromParser); NodeCast::from_temporary(document) @@ -1551,67 +1552,67 @@ impl Node { }; let element = Element::create(name, element.prefix().as_ref().map(|p| p.as_slice().into_string()), - *document, ElementCreator::ScriptCreated); + document.r(), ElementCreator::ScriptCreated); NodeCast::from_temporary(element) }, NodeTypeId::Text => { let text: JSRef<Text> = TextCast::to_ref(node).unwrap(); - let text = Text::new(text.characterdata().data().clone(), *document); + let text = Text::new(text.characterdata().data().clone(), document.r()); NodeCast::from_temporary(text) }, NodeTypeId::ProcessingInstruction => { let pi: JSRef<ProcessingInstruction> = ProcessingInstructionCast::to_ref(node).unwrap(); let pi = ProcessingInstruction::new(pi.target().clone(), - pi.characterdata().data().clone(), *document); + pi.characterdata().data().clone(), document.r()); NodeCast::from_temporary(pi) }, }.root(); // Step 3. - let document = match DocumentCast::to_ref(*copy) { + let document = match DocumentCast::to_ref(copy.r()) { Some(doc) => doc, - None => *document, + None => document.r(), }; - assert!(*copy.owner_doc().root() == document); + assert!(copy.r().owner_doc().root().r() == document); // Step 4 (some data already copied in step 2). match node.type_id() { NodeTypeId::Document => { let node_doc: JSRef<Document> = DocumentCast::to_ref(node).unwrap(); - let copy_doc: JSRef<Document> = DocumentCast::to_ref(*copy).unwrap(); + let copy_doc: JSRef<Document> = DocumentCast::to_ref(copy.r()).unwrap(); copy_doc.set_encoding_name(node_doc.encoding_name().clone()); copy_doc.set_quirks_mode(node_doc.quirks_mode()); }, NodeTypeId::Element(..) => { let node_elem: JSRef<Element> = ElementCast::to_ref(node).unwrap(); - let copy_elem: JSRef<Element> = ElementCast::to_ref(*copy).unwrap(); + let copy_elem: JSRef<Element> = ElementCast::to_ref(copy.r()).unwrap(); // FIXME: https://github.com/mozilla/servo/issues/1737 let window = document.window().root(); for attr in node_elem.attrs().iter().map(|attr| attr.root()) { copy_elem.attrs_mut().push_unrooted( - &Attr::new(*window, - attr.local_name().clone(), attr.value().clone(), - attr.name().clone(), attr.namespace().clone(), - attr.prefix().clone(), Some(copy_elem))); + &Attr::new(window.r(), + attr.r().local_name().clone(), attr.r().value().clone(), + attr.r().name().clone(), attr.r().namespace().clone(), + attr.r().prefix().clone(), Some(copy_elem))); } }, _ => () } // Step 5: cloning steps. - vtable_for(&node).cloning_steps(*copy, maybe_doc, clone_children); + vtable_for(&node).cloning_steps(copy.r(), maybe_doc, clone_children); // Step 6. if clone_children == CloneChildrenFlag::CloneChildren { for child in node.children() { let child_copy = Node::clone(child, Some(document), clone_children).root(); - let _inserted_node = Node::pre_insert(*child_copy, *copy, None); + let _inserted_node = Node::pre_insert(child_copy.r(), copy.r(), None); } } // Step 7. - Temporary::from_rooted(*copy) + Temporary::from_rooted(copy.r()) } /// Sends layout data, if any, back to the layout task to be destroyed. @@ -1708,7 +1709,7 @@ impl<'a> NodeMethods for JSRef<'a, Node> { self.parent_node.get() .and_then(|parent| { let parent = parent.root(); - ElementCast::to_ref(*parent).map(|elem| { + ElementCast::to_ref(parent.r()).map(|elem| { Temporary::from_rooted(elem) }) }) @@ -1723,8 +1724,8 @@ impl<'a> NodeMethods for JSRef<'a, Node> { fn ChildNodes(self) -> Temporary<NodeList> { self.child_list.or_init(|| { let doc = self.owner_doc().root(); - let window = doc.window().root(); - NodeList::new_child_list(*window, self) + let window = doc.r().window().root(); + NodeList::new_child_list(window.r(), self) }) } @@ -1807,7 +1808,7 @@ impl<'a> NodeMethods for JSRef<'a, Node> { None } else { let document = self.owner_doc().root(); - Some(NodeCast::from_temporary(document.CreateTextNode(value))) + Some(NodeCast::from_temporary(document.r().CreateTextNode(value))) }.root(); // Step 3. @@ -1821,7 +1822,7 @@ impl<'a> NodeMethods for JSRef<'a, Node> { // Notify the document that the content of this node is different let document = self.owner_doc().root(); - document.content_changed(self, NodeDamage::OtherNodeDamage); + document.r().content_changed(self, NodeDamage::OtherNodeDamage); } NodeTypeId::DocumentType | NodeTypeId::Document => {} @@ -1942,7 +1943,7 @@ impl<'a> NodeMethods for JSRef<'a, Node> { // Step 9. let document = document_from_node(self).root(); - Node::adopt(node, *document); + Node::adopt(node, document.r()); { // Step 10. @@ -2044,9 +2045,9 @@ impl<'a> NodeMethods for JSRef<'a, Node> { assert!(element.attrs().len() == other_element.attrs().len()); element.attrs().iter().map(|attr| attr.root()).all(|attr| { other_element.attrs().iter().map(|attr| attr.root()).any(|other_attr| { - (*attr.namespace() == *other_attr.namespace()) && - (attr.local_name() == other_attr.local_name()) && - (attr.value().as_slice() == other_attr.value().as_slice()) + (*attr.r().namespace() == *other_attr.r().namespace()) && + (attr.r().local_name() == other_attr.r().local_name()) && + (attr.r().value().as_slice() == other_attr.r().value().as_slice()) }) }) } @@ -2183,7 +2184,7 @@ pub fn document_from_node<T: NodeBase+Reflectable>(derived: JSRef<T>) -> Tempora pub fn window_from_node<T: NodeBase+Reflectable>(derived: JSRef<T>) -> Temporary<Window> { let document = document_from_node(derived).root(); - document.window() + document.r().window() } impl<'a> VirtualMethods for JSRef<'a, Node> { @@ -2279,12 +2280,12 @@ impl<'a> style::TNode<'a, JSRef<'a, Element>> for JSRef<'a, Node> { match attr.namespace { style::NamespaceConstraint::Specific(ref ns) => { self.as_element().get_attribute(ns.clone(), name).root() - .map_or(false, |attr| test(attr.value().as_slice())) + .map_or(false, |attr| test(attr.r().value().as_slice())) }, style::NamespaceConstraint::Any => { self.as_element().get_attributes(name).iter() .map(|attr| attr.root()) - .any(|attr| test(attr.value().as_slice())) + .any(|attr| test(attr.r().value().as_slice())) } } } @@ -2338,7 +2339,7 @@ impl<'a> DisabledStateHelpers for JSRef<'a, Node> { fn check_parent_disabled_state_for_option(self) { if self.get_disabled_state() { return; } match self.parent_node().root() { - Some(ref parent) if parent.is_htmloptgroupelement() && parent.get_disabled_state() => { + Some(ref parent) if parent.r().is_htmloptgroupelement() && parent.r().get_disabled_state() => { self.set_disabled_state(true); self.set_enabled_state(false); }, diff --git a/components/script/dom/nodelist.rs b/components/script/dom/nodelist.rs index 5714764eeaf..7a1bcd8ecac 100644 --- a/components/script/dom/nodelist.rs +++ b/components/script/dom/nodelist.rs @@ -52,7 +52,7 @@ impl<'a> NodeListMethods for JSRef<'a, NodeList> { NodeListType::Simple(ref elems) => elems.len() as u32, NodeListType::Children(ref node) => { let node = node.root(); - node.children().count() as u32 + node.r().children().count() as u32 } } } @@ -63,8 +63,8 @@ impl<'a> NodeListMethods for JSRef<'a, NodeList> { NodeListType::Simple(ref elems) => Some(Temporary::new(elems[index as uint].clone())), NodeListType::Children(ref node) => { let node = node.root(); - node.children().nth(index as uint) - .map(|child| Temporary::from_rooted(child)) + node.r().children().nth(index as uint) + .map(|child| Temporary::from_rooted(child)) } } } diff --git a/components/script/dom/performance.rs b/components/script/dom/performance.rs index ecc15868939..31f0fd3b31e 100644 --- a/components/script/dom/performance.rs +++ b/components/script/dom/performance.rs @@ -49,7 +49,7 @@ impl<'a> PerformanceMethods for JSRef<'a, Performance> { // https://dvcs.w3.org/hg/webperf/raw-file/tip/specs/HighResolutionTime/Overview.html#dom-performance-now fn Now(self) -> DOMHighResTimeStamp { - let navStart = self.timing.root().NavigationStartPrecise(); + let navStart = self.timing.root().r().NavigationStartPrecise(); (time::precise_time_ns() as f64 - navStart) * 1000000u as DOMHighResTimeStamp } } diff --git a/components/script/dom/progressevent.rs b/components/script/dom/progressevent.rs index 4a05c1658da..758db362842 100644 --- a/components/script/dom/progressevent.rs +++ b/components/script/dom/progressevent.rs @@ -42,9 +42,9 @@ impl ProgressEvent { let ev = reflect_dom_object(box ProgressEvent::new_inherited(length_computable, loaded, total), global, ProgressEventBinding::Wrap).root(); - let event: JSRef<Event> = EventCast::from_ref(*ev); + let event: JSRef<Event> = EventCast::from_ref(ev.r()); event.InitEvent(type_, can_bubble, cancelable); - Temporary::from_rooted(*ev) + Temporary::from_rooted(ev.r()) } pub fn Constructor(global: &GlobalRef, type_: DOMString, diff --git a/components/script/dom/range.rs b/components/script/dom/range.rs index 74cf1f8bafd..0a7714c5b70 100644 --- a/components/script/dom/range.rs +++ b/components/script/dom/range.rs @@ -26,13 +26,13 @@ impl Range { pub fn new(document: JSRef<Document>) -> Temporary<Range> { let window = document.window().root(); reflect_dom_object(box Range::new_inherited(), - GlobalRef::Window(*window), + GlobalRef::Window(window.r()), RangeBinding::Wrap) } pub fn Constructor(global: &GlobalRef) -> Fallible<Temporary<Range>> { let document = global.as_window().Document().root(); - Ok(Range::new(*document)) + Ok(Range::new(document.r())) } } diff --git a/components/script/dom/servohtmlparser.rs b/components/script/dom/servohtmlparser.rs index 30807adad02..d06a50c04c8 100644 --- a/components/script/dom/servohtmlparser.rs +++ b/components/script/dom/servohtmlparser.rs @@ -72,7 +72,8 @@ impl ServoHTMLParser { tokenizer: DOMRefCell::new(tok), }; - reflect_dom_object(box parser, GlobalRef::Window(*window), ServoHTMLParserBinding::Wrap) + reflect_dom_object(box parser, GlobalRef::Window(window.r()), + ServoHTMLParserBinding::Wrap) } #[inline] diff --git a/components/script/dom/text.rs b/components/script/dom/text.rs index 0a925309dad..72082e11805 100644 --- a/components/script/dom/text.rs +++ b/components/script/dom/text.rs @@ -40,7 +40,7 @@ impl Text { pub fn Constructor(global: &GlobalRef, text: DOMString) -> Fallible<Temporary<Text>> { let document = global.as_window().Document().root(); - Ok(Text::new(text, *document)) + Ok(Text::new(text, document.r())) } #[inline] diff --git a/components/script/dom/treewalker.rs b/components/script/dom/treewalker.rs index 5551ec1e211..252d82acbe9 100644 --- a/components/script/dom/treewalker.rs +++ b/components/script/dom/treewalker.rs @@ -47,7 +47,7 @@ impl TreeWalker { filter: Filter) -> Temporary<TreeWalker> { let window = document.window().root(); reflect_dom_object(box TreeWalker::new_inherited(root_node, what_to_show, filter), - GlobalRef::Window(*window), + GlobalRef::Window(window.r()), TreeWalkerBinding::Wrap) } diff --git a/components/script/dom/uievent.rs b/components/script/dom/uievent.rs index a5a867fd222..00a17063e9f 100644 --- a/components/script/dom/uievent.rs +++ b/components/script/dom/uievent.rs @@ -53,8 +53,8 @@ impl UIEvent { view: Option<JSRef<Window>>, detail: i32) -> Temporary<UIEvent> { let ev = UIEvent::new_uninitialized(window).root(); - ev.InitUIEvent(type_, can_bubble, cancelable, view, detail); - Temporary::from_rooted(*ev) + ev.r().InitUIEvent(type_, can_bubble, cancelable, view, detail); + Temporary::from_rooted(ev.r()) } pub fn Constructor(global: &GlobalRef, diff --git a/components/script/dom/urlsearchparams.rs b/components/script/dom/urlsearchparams.rs index d472d294d5f..fdddc7219c7 100644 --- a/components/script/dom/urlsearchparams.rs +++ b/components/script/dom/urlsearchparams.rs @@ -52,7 +52,7 @@ impl URLSearchParams { Some(eURLSearchParams(u)) => { let u = u.root(); let mut map = usp.data.borrow_mut(); - *map = u.data.borrow().clone(); + *map = u.r().data.borrow().clone(); }, None => {} } diff --git a/components/script/dom/worker.rs b/components/script/dom/worker.rs index d6e4391cbfd..b222311c37a 100644 --- a/components/script/dom/worker.rs +++ b/components/script/dom/worker.rs @@ -71,20 +71,20 @@ impl Worker { let (sender, receiver) = channel(); let worker = Worker::new(global, sender.clone()).root(); - let worker_ref = Trusted::new(global.get_cx(), *worker, global.script_chan()); + let worker_ref = Trusted::new(global.get_cx(), worker.r(), global.script_chan()); DedicatedWorkerGlobalScope::run_worker_scope( worker_url, worker_ref, resource_task, global.script_chan(), sender, receiver); - Ok(Temporary::from_rooted(*worker)) + Ok(Temporary::from_rooted(worker.r())) } pub fn handle_message(address: TrustedWorkerAddress, data: *mut u64, nbytes: size_t) { let worker = address.to_temporary().root(); - let global = worker.global.root(); + let global = worker.r().global.root(); let mut message = UndefinedValue(); unsafe { @@ -94,7 +94,7 @@ impl Worker { ptr::null(), ptr::null_mut()) != 0); } - let target: JSRef<EventTarget> = EventTargetCast::from_ref(*worker); + let target: JSRef<EventTarget> = EventTargetCast::from_ref(worker.r()); MessageEvent::dispatch_jsval(target, global.root_ref(), message); } } diff --git a/components/script/dom/xmlhttprequest.rs b/components/script/dom/xmlhttprequest.rs index c8e2fbdd5a5..a0316f51ddf 100644 --- a/components/script/dom/xmlhttprequest.rs +++ b/components/script/dom/xmlhttprequest.rs @@ -201,7 +201,7 @@ impl XMLHttpRequest { pub fn handle_progress(addr: TrustedXHRAddress, progress: XHRProgress) { let xhr = addr.to_temporary().root(); - xhr.process_partial_response(progress); + xhr.r().process_partial_response(progress); } fn fetch(fetch_type: &SyncOrAsync, resource_task: ResourceTask, @@ -769,7 +769,7 @@ impl<'a> PrivateXMLHttpRequestHelpers for JSRef<'a, XMLHttpRequest> { EventBubbles::DoesNotBubble, EventCancelable::Cancelable).root(); let target: JSRef<EventTarget> = EventTargetCast::from_ref(self); - target.dispatch_event(*event); + target.dispatch_event(event.r()); } fn process_partial_response(self, progress: XHRProgress) { @@ -908,7 +908,7 @@ impl<'a> PrivateXMLHttpRequestHelpers for JSRef<'a, XMLHttpRequest> { } else { EventTargetCast::from_ref(self) }; - let event: JSRef<Event> = EventCast::from_ref(*progressevent); + let event: JSRef<Event> = EventCast::from_ref(progressevent.r()); target.dispatch_event(event); } @@ -1008,7 +1008,7 @@ impl Extractable for SendParam { let encoding = UTF_8 as EncodingRef; match *self { eString(ref s) => encoding.encode(s.as_slice(), EncoderTrap::Replace).unwrap(), - eURLSearchParams(ref usp) => usp.root().serialize(None) // Default encoding is UTF8 + eURLSearchParams(ref usp) => usp.root().r().serialize(None) // Default encoding is UTF8 } } } diff --git a/components/script/page.rs b/components/script/page.rs index 3c6dcb43f91..979f49f6681 100644 --- a/components/script/page.rs +++ b/components/script/page.rs @@ -166,7 +166,7 @@ impl Page { pub fn flush_layout(&self, goal: ReflowGoal, query: ReflowQueryType) { let frame = self.frame(); let window = frame.as_ref().unwrap().window.root(); - self.reflow(goal, window.control_chan().clone(), &mut **window.compositor(), query); + self.reflow(goal, window.r().control_chan().clone(), &mut **window.r().compositor(), query); } pub fn layout(&self) -> &LayoutRPC { @@ -238,7 +238,7 @@ impl Page { Some(ref frame) => { let window = frame.window.root(); let document = frame.document.root(); - window.compositor().set_title(self.id, Some(document.Title())); + window.r().compositor().set_title(self.id, Some(document.r().Title())); } } } @@ -246,7 +246,7 @@ impl Page { pub fn dirty_all_nodes(&self) { match *self.frame.borrow() { None => {} - Some(ref frame) => frame.document.root().dirty_all_nodes(), + Some(ref frame) => frame.document.root().r().dirty_all_nodes(), } } } @@ -344,7 +344,7 @@ impl Page { let root = match *self.frame() { None => return, Some(ref frame) => { - frame.document.root().GetDocumentElement() + frame.document.root().r().GetDocumentElement() } }; @@ -355,7 +355,7 @@ impl Page { debug!("script: performing reflow for goal {}", goal); - let root: JSRef<Node> = NodeCast::from_ref(*root); + let root: JSRef<Node> = NodeCast::from_ref(root.r()); if !root.get_has_dirty_descendants() { debug!("root has no dirty descendants; avoiding reflow"); return @@ -401,17 +401,17 @@ impl Page { /// Attempt to find a named element in this page's document. pub fn find_fragment_node(&self, fragid: DOMString) -> Option<Temporary<Element>> { let document = self.frame().as_ref().unwrap().document.root(); - document.find_fragment_node(fragid) + document.r().find_fragment_node(fragid) } pub fn hit_test(&self, point: &Point2D<f32>) -> Option<UntrustedNodeAddress> { let frame = self.frame(); let document = frame.as_ref().unwrap().document.root(); - let root = match document.GetDocumentElement().root() { + let root = match document.r().GetDocumentElement().root() { None => return None, Some(root) => root, }; - let root: JSRef<Node> = NodeCast::from_ref(*root); + let root: JSRef<Node> = NodeCast::from_ref(root.r()); let address = match self.layout().hit_test(root.to_trusted_node_address(), *point) { Ok(HitTestResponse(node_address)) => { Some(node_address) @@ -427,11 +427,11 @@ impl Page { pub fn get_nodes_under_mouse(&self, point: &Point2D<f32>) -> Option<Vec<UntrustedNodeAddress>> { let frame = self.frame(); let document = frame.as_ref().unwrap().document.root(); - let root = match document.GetDocumentElement().root() { + let root = match document.r().GetDocumentElement().root() { None => return None, Some(root) => root, }; - let root: JSRef<Node> = NodeCast::from_ref(*root); + let root: JSRef<Node> = NodeCast::from_ref(root.r()); let address = match self.layout().mouse_over(root.to_trusted_node_address(), *point) { Ok(MouseOverResponse(node_address)) => { Some(node_address) diff --git a/components/script/parse/html.rs b/components/script/parse/html.rs index ab30802d16e..eb97d0893a4 100644 --- a/components/script/parse/html.rs +++ b/components/script/parse/html.rs @@ -46,7 +46,7 @@ impl SinkHelpers for servohtmlparser::Sink { AppendNode(n) => Temporary::new(unsafe { JS::from_trusted_node_address(n) }), AppendText(t) => { let doc = self.document.root(); - let text = Text::new(t, *doc); + let text = Text::new(t, doc.r()); NodeCast::from_temporary(text) } } @@ -56,7 +56,7 @@ impl SinkHelpers for servohtmlparser::Sink { impl<'a> TreeSink<TrustedNodeAddress> for servohtmlparser::Sink { fn get_document(&mut self) -> TrustedNodeAddress { let doc = self.document.root(); - let node: JSRef<Node> = NodeCast::from_ref(*doc); + let node: JSRef<Node> = NodeCast::from_ref(doc.r()); node.to_trusted_node_address() } @@ -66,7 +66,7 @@ impl<'a> TreeSink<TrustedNodeAddress> for servohtmlparser::Sink { fn elem_name(&self, target: TrustedNodeAddress) -> QualName { let node: Root<Node> = unsafe { JS::from_trusted_node_address(target).root() }; - let elem: JSRef<Element> = ElementCast::to_ref(*node) + let elem: JSRef<Element> = ElementCast::to_ref(node.r()) .expect("tried to get name of non-Element in HTML parsing"); QualName { ns: elem.namespace().clone(), @@ -77,22 +77,22 @@ impl<'a> TreeSink<TrustedNodeAddress> for servohtmlparser::Sink { fn create_element(&mut self, name: QualName, attrs: Vec<Attribute>) -> TrustedNodeAddress { let doc = self.document.root(); - let elem = Element::create(name, None, *doc, + let elem = Element::create(name, None, doc.r(), ElementCreator::ParserCreated).root(); for attr in attrs.into_iter() { - elem.set_attribute_from_parser(attr.name, attr.value, None); + elem.r().set_attribute_from_parser(attr.name, attr.value, None); } - let node: JSRef<Node> = NodeCast::from_ref(*elem); + let node: JSRef<Node> = NodeCast::from_ref(elem.r()); node.to_trusted_node_address() } fn create_comment(&mut self, text: String) -> TrustedNodeAddress { let doc = self.document.root(); - let comment = Comment::new(text, *doc); + let comment = Comment::new(text, doc.r()); let node: Root<Node> = NodeCast::from_temporary(comment).root(); - node.to_trusted_node_address() + node.r().to_trusted_node_address() } fn append_before_sibling(&mut self, @@ -100,13 +100,13 @@ impl<'a> TreeSink<TrustedNodeAddress> for servohtmlparser::Sink { new_node: NodeOrText<TrustedNodeAddress>) -> Result<(), NodeOrText<TrustedNodeAddress>> { // If there is no parent, return the node to the parser. let sibling: Root<Node> = unsafe { JS::from_trusted_node_address(sibling).root() }; - let parent = match sibling.parent_node() { + let parent = match sibling.r().parent_node() { Some(p) => p.root(), None => return Err(new_node), }; let child = self.get_or_create(new_node).root(); - assert!(parent.InsertBefore(*child, Some(*sibling)).is_ok()); + assert!(parent.r().InsertBefore(child.r(), Some(sibling.r())).is_ok()); Ok(()) } @@ -116,7 +116,7 @@ impl<'a> TreeSink<TrustedNodeAddress> for servohtmlparser::Sink { fn set_quirks_mode(&mut self, mode: QuirksMode) { let doc = self.document.root(); - doc.set_quirks_mode(mode); + doc.r().set_quirks_mode(mode); } fn append(&mut self, parent: TrustedNodeAddress, child: NodeOrText<TrustedNodeAddress>) { @@ -124,21 +124,21 @@ impl<'a> TreeSink<TrustedNodeAddress> for servohtmlparser::Sink { let child = self.get_or_create(child).root(); // FIXME(#3701): Use a simpler algorithm and merge adjacent text nodes - assert!(parent.AppendChild(*child).is_ok()); + assert!(parent.r().AppendChild(child.r()).is_ok()); } fn append_doctype_to_document(&mut self, name: String, public_id: String, system_id: String) { let doc = self.document.root(); - let doc_node: JSRef<Node> = NodeCast::from_ref(*doc); - let doctype = DocumentType::new(name, Some(public_id), Some(system_id), *doc); + let doc_node: JSRef<Node> = NodeCast::from_ref(doc.r()); + let doctype = DocumentType::new(name, Some(public_id), Some(system_id), doc.r()); let node: Root<Node> = NodeCast::from_temporary(doctype).root(); - assert!(doc_node.AppendChild(*node).is_ok()); + assert!(doc_node.AppendChild(node.r()).is_ok()); } fn add_attrs_if_missing(&mut self, target: TrustedNodeAddress, attrs: Vec<Attribute>) { let node: Root<Node> = unsafe { JS::from_trusted_node_address(target).root() }; - let elem: JSRef<Element> = ElementCast::to_ref(*node) + let elem: JSRef<Element> = ElementCast::to_ref(node.r()) .expect("tried to set attrs on non-Element in HTML parsing"); for attr in attrs.into_iter() { elem.set_attribute_from_parser(attr.name, attr.value, None); @@ -151,13 +151,13 @@ impl<'a> TreeSink<TrustedNodeAddress> for servohtmlparser::Sink { fn mark_script_already_started(&mut self, node: TrustedNodeAddress) { let node: Root<Node> = unsafe { JS::from_trusted_node_address(node).root() }; - let script: Option<JSRef<HTMLScriptElement>> = HTMLScriptElementCast::to_ref(*node); + let script: Option<JSRef<HTMLScriptElement>> = HTMLScriptElementCast::to_ref(node.r()); script.map(|script| script.mark_already_started()); } fn complete_script(&mut self, node: TrustedNodeAddress) { let node: Root<Node> = unsafe { JS::from_trusted_node_address(node).root() }; - let script: Option<JSRef<HTMLScriptElement>> = HTMLScriptElementCast::to_ref(*node); + let script: Option<JSRef<HTMLScriptElement>> = HTMLScriptElementCast::to_ref(node.r()); script.map(|script| script.prepare()); } } @@ -166,7 +166,7 @@ pub fn parse_html(document: JSRef<Document>, input: HTMLInput, url: &Url) { let parser = ServoHTMLParser::new(Some(url.clone()), document).root(); - let parser: JSRef<ServoHTMLParser> = *parser; + let parser: JSRef<ServoHTMLParser> = parser.r(); let nested_parse = task_state::get().contains(task_state::IN_HTML_PARSER); if !nested_parse { diff --git a/components/script/script_task.rs b/components/script/script_task.rs index c1476780d61..dd190e847a0 100644 --- a/components/script/script_task.rs +++ b/components/script/script_task.rs @@ -654,7 +654,7 @@ impl ScriptTask { pipeline ID not associated with this script task. This is a bug."); let frame = page.frame(); let window = frame.as_ref().unwrap().window.root(); - window.handle_fire_timer(timer_id); + window.r().handle_fire_timer(timer_id); } /// Handles a notification that reflow completed. @@ -787,11 +787,11 @@ impl ScriptTask { } else { url.clone() }; - let document = Document::new(*window, Some(doc_url.clone()), + let document = Document::new(window.r(), Some(doc_url.clone()), IsHTMLDocument::HTMLDocument, None, DocumentSource::FromParser).root(); - window.init_browser_context(*document); + window.r().init_browser_context(document.r()); self.compositor.borrow_mut().set_ready_state(pipeline_id, Loading); @@ -799,8 +799,8 @@ impl ScriptTask { // Create the root frame. let mut frame = page.mut_frame(); *frame = Some(Frame { - document: JS::from_rooted(*document), - window: JS::from_rooted(*window), + document: JS::from_rooted(document.r()), + window: JS::from_rooted(window.r()), }); } @@ -820,7 +820,7 @@ impl ScriptTask { load_response.metadata.headers.as_ref().map(|headers| { headers.get().map(|&LastModified(ref tm)| { - document.set_last_modified(dom_last_modified(tm)); + document.r().set_last_modified(dom_last_modified(tm)); }); }); @@ -836,22 +836,22 @@ impl ScriptTask { (HTMLInput::InputUrl(load_response), final_url) } else { let evalstr = load_data.url.non_relative_scheme_data().unwrap(); - let jsval = window.evaluate_js_with_result(evalstr); + let jsval = window.r().evaluate_js_with_result(evalstr); let strval = FromJSValConvertible::from_jsval(self.get_cx(), jsval, StringificationBehavior::Empty); (HTMLInput::InputString(strval.unwrap_or("".into_string())), doc_url) }; - parse_html(*document, parser_input, &final_url); + parse_html(document.r(), parser_input, &final_url); - document.set_ready_state(DocumentReadyState::Interactive); + document.r().set_ready_state(DocumentReadyState::Interactive); self.compositor.borrow_mut().set_ready_state(pipeline_id, PerformingLayout); // Kick off the initial reflow of the page. debug!("kicking off initial reflow of {}", final_url); - document.content_changed(NodeCast::from_ref(*document), - NodeDamage::OtherNodeDamage); - window.flush_layout(ReflowGoal::ForDisplay, ReflowQueryType::NoQuery); + document.r().content_changed(NodeCast::from_ref(document.r()), + NodeDamage::OtherNodeDamage); + window.r().flush_layout(ReflowGoal::ForDisplay, ReflowQueryType::NoQuery); { // No more reflow required @@ -860,24 +860,24 @@ impl ScriptTask { } // https://html.spec.whatwg.org/multipage/#the-end step 4 - let event = Event::new(GlobalRef::Window(*window), "DOMContentLoaded".into_string(), + let event = Event::new(GlobalRef::Window(window.r()), "DOMContentLoaded".into_string(), EventBubbles::DoesNotBubble, EventCancelable::NotCancelable).root(); - let doctarget: JSRef<EventTarget> = EventTargetCast::from_ref(*document); - let _ = doctarget.DispatchEvent(*event); + let doctarget: JSRef<EventTarget> = EventTargetCast::from_ref(document.r()); + let _ = doctarget.DispatchEvent(event.r()); // We have no concept of a document loader right now, so just dispatch the // "load" event as soon as we've finished executing all scripts parsed during // the initial load. // https://html.spec.whatwg.org/multipage/#the-end step 7 - document.set_ready_state(DocumentReadyState::Complete); + document.r().set_ready_state(DocumentReadyState::Complete); - let event = Event::new(GlobalRef::Window(*window), "load".into_string(), + let event = Event::new(GlobalRef::Window(window.r()), "load".into_string(), EventBubbles::DoesNotBubble, EventCancelable::NotCancelable).root(); - let wintarget: JSRef<EventTarget> = EventTargetCast::from_ref(*window); - let _ = wintarget.dispatch_event_with_target(doctarget, *event); + let wintarget: JSRef<EventTarget> = EventTargetCast::from_ref(window.r()); + let _ = wintarget.dispatch_event_with_target(doctarget, event.r()); *page.fragment_name.borrow_mut() = final_url.fragment.clone(); @@ -889,7 +889,7 @@ impl ScriptTask { None => {} Some(ref chan) => { let page_info = DevtoolsPageInfo { - title: document.Title(), + title: document.r().Title(), url: final_url }; chan.send(NewGlobal(pipeline_id, self.devtools_sender.clone(), @@ -945,7 +945,8 @@ impl ScriptTask { let page = get_page(&*self.page.borrow(), pipeline_id); let frame = page.frame(); let document = frame.as_ref().unwrap().document.root(); - document.content_changed(*node_to_dirty, NodeDamage::OtherNodeDamage); + document.r().content_changed(node_to_dirty.r(), + NodeDamage::OtherNodeDamage); } self.handle_reflow_event(pipeline_id); @@ -975,14 +976,14 @@ impl ScriptTask { let page = get_page(&*self.page.borrow(), pipeline_id); let frame = page.frame(); let window = frame.as_ref().unwrap().window.root(); - let doc = window.Document().root(); - let focused = doc.get_focused_element().root(); - let body = doc.GetBody().root(); + let doc = window.r().Document().root(); + let focused = doc.r().get_focused_element().root(); + let body = doc.r().GetBody().root(); let target: JSRef<EventTarget> = match (&focused, &body) { - (&Some(ref focused), _) => EventTargetCast::from_ref(**focused), - (&None, &Some(ref body)) => EventTargetCast::from_ref(**body), - (&None, &None) => EventTargetCast::from_ref(*window), + (&Some(ref focused), _) => EventTargetCast::from_ref(focused.r()), + (&None, &Some(ref body)) => EventTargetCast::from_ref(body.r()), + (&None, &None) => EventTargetCast::from_ref(window.r()), }; let ctrl = modifiers.contains(CONTROL); @@ -999,26 +1000,28 @@ impl ScriptTask { let props = KeyboardEvent::key_properties(key, modifiers); - let keyevent = KeyboardEvent::new(*window, ev_type, true, true, Some(*window), 0, + let keyevent = KeyboardEvent::new(window.r(), ev_type, true, true, + Some(window.r()), 0, props.key.into_string(), props.code.into_string(), props.location, is_repeating, is_composing, ctrl, alt, shift, meta, None, props.key_code).root(); - let event = EventCast::from_ref(*keyevent); + let event = EventCast::from_ref(keyevent.r()); let _ = target.DispatchEvent(event); let mut prevented = event.DefaultPrevented(); // https://dvcs.w3.org/hg/dom3events/raw-file/tip/html/DOM3-Events.html#keys-cancelable-keys if state != KeyState::Released && props.is_printable() && !prevented { // https://dvcs.w3.org/hg/dom3events/raw-file/tip/html/DOM3-Events.html#keypress-event-order - let event = KeyboardEvent::new(*window, "keypress".into_string(), true, true, Some(*window), + let event = KeyboardEvent::new(window.r(), "keypress".into_string(), + true, true, Some(window.r()), 0, props.key.into_string(), props.code.into_string(), props.location, is_repeating, is_composing, ctrl, alt, shift, meta, props.char_code, 0).root(); - let _ = target.DispatchEvent(EventCast::from_ref(*event)); + let _ = target.DispatchEvent(EventCast::from_ref(event.r())); - let ev = EventCast::from_ref(*event); + let ev = EventCast::from_ref(event.r()); prevented = ev.DefaultPrevented(); // TODO: if keypress event is canceled, prevent firing input events } @@ -1044,7 +1047,7 @@ impl ScriptTask { _ => () } - window.flush_layout(ReflowGoal::ForDisplay, ReflowQueryType::NoQuery); + window.r().flush_layout(ReflowGoal::ForDisplay, ReflowQueryType::NoQuery); } /// The entry point for content to notify that a new load has been requested @@ -1060,7 +1063,7 @@ impl ScriptTask { let page = get_page(&*self.page.borrow(), pipeline_id); match page.find_fragment_node(url.fragment.unwrap()).root() { Some(node) => { - self.scroll_fragment_point(pipeline_id, *node); + self.scroll_fragment_point(pipeline_id, node.r()); } None => {} } @@ -1084,7 +1087,7 @@ impl ScriptTask { .and_then(|name| page.find_fragment_node(name)) .root(); match fragment_node { - Some(node) => self.scroll_fragment_point(pipeline_id, *node), + Some(node) => self.scroll_fragment_point(pipeline_id, node.r()), None => {} } @@ -1095,13 +1098,13 @@ impl ScriptTask { Some(window) => { // http://dev.w3.org/csswg/cssom-view/#resizing-viewports // https://dvcs.w3.org/hg/dom3events/raw-file/tip/html/DOM3-Events.html#event-type-resize - let uievent = UIEvent::new(window.clone(), + let uievent = UIEvent::new(window.r(), "resize".into_string(), false, - false, Some(window.clone()), + false, Some(window.r()), 0i32).root(); - let event: JSRef<Event> = EventCast::from_ref(*uievent); + let event: JSRef<Event> = EventCast::from_ref(uievent.r()); - let wintarget: JSRef<EventTarget> = EventTargetCast::from_ref(*window); + let wintarget: JSRef<EventTarget> = EventTargetCast::from_ref(window.r()); wintarget.dispatch_event(event); } None => () @@ -1128,9 +1131,9 @@ impl ScriptTask { node::from_untrusted_node_address( self.js_runtime.ptr, node_address).root(); - let maybe_node = match ElementCast::to_ref(*temp_node) { + let maybe_node = match ElementCast::to_ref(temp_node.r()) { Some(element) => Some(element), - None => temp_node.ancestors().filter_map(ElementCast::to_ref).next(), + None => temp_node.r().ancestors().filter_map(ElementCast::to_ref).next(), }; match maybe_node { @@ -1142,21 +1145,21 @@ impl ScriptTask { match *page.frame() { Some(ref frame) => { let window = frame.window.root(); - let doc = window.Document().root(); - doc.begin_focus_transaction(); + let doc = window.r().Document().root(); + doc.r().begin_focus_transaction(); let event = - Event::new(GlobalRef::Window(*window), + Event::new(GlobalRef::Window(window.r()), "click".into_string(), EventBubbles::Bubbles, EventCancelable::Cancelable).root(); // https://dvcs.w3.org/hg/dom3events/raw-file/tip/html/DOM3-Events.html#trusted-events - event.set_trusted(true); + event.r().set_trusted(true); // https://html.spec.whatwg.org/multipage/interaction.html#run-authentic-click-activation-steps - el.authentic_click_activation(*event); + el.authentic_click_activation(event.r()); - doc.commit_focus_transaction(); - window.flush_layout(ReflowGoal::ForDisplay, ReflowQueryType::NoQuery); + doc.r().commit_focus_transaction(); + window.r().flush_layout(ReflowGoal::ForDisplay, ReflowQueryType::NoQuery); } None => {} } @@ -1182,7 +1185,7 @@ impl ScriptTask { Some(ref mut mouse_over_targets) => { for node in mouse_over_targets.iter_mut() { let node = node.root(); - node.set_hover_state(false); + node.r().set_hover_state(false); } } None => {} @@ -1198,19 +1201,19 @@ impl ScriptTask { let x = point.x.to_i32().unwrap_or(0); let y = point.y.to_i32().unwrap_or(0); - let mouse_event = MouseEvent::new(*window, + let mouse_event = MouseEvent::new(window.r(), "mousemove".into_string(), true, true, - Some(*window), + Some(window.r()), 0i32, x, y, x, y, false, false, false, false, 0i16, None).root(); - let event: JSRef<Event> = EventCast::from_ref(*mouse_event); - let target: JSRef<EventTarget> = EventTargetCast::from_ref(*top_most_node); + let event: JSRef<Event> = EventCast::from_ref(mouse_event.r()); + let target: JSRef<EventTarget> = EventTargetCast::from_ref(top_most_node.r()); target.dispatch_event(event); } } |