From 89a0c004d574c69f9ca03bde2ed341cc51b21f0f Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Fri, 14 Nov 2014 15:28:17 +0100 Subject: [PATCH 1/3] Store an Atom for Attr::prefix. --- components/script/dom/attr.rs | 8 ++++---- components/script/dom/node.rs | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/components/script/dom/attr.rs b/components/script/dom/attr.rs index c25c9d4d611..aef9d65e5ff 100644 --- a/components/script/dom/attr.rs +++ b/components/script/dom/attr.rs @@ -101,7 +101,7 @@ pub struct Attr { value: DOMRefCell, name: Atom, namespace: Namespace, - prefix: Option, + prefix: Option, /// the element that owns this attribute. owner: MutNullableHeap>, @@ -116,7 +116,7 @@ impl Attr { value: DOMRefCell::new(value), name: name, namespace: namespace, - prefix: prefix, + prefix: prefix.map(|p| Atom::from_slice(&p)), owner: MutNullableHeap::new(owner.map(JS::from_rooted)), } } @@ -141,7 +141,7 @@ impl Attr { } #[inline] - pub fn prefix<'a>(&'a self) -> &'a Option { + pub fn prefix<'a>(&'a self) -> &'a Option { &self.prefix } } @@ -205,7 +205,7 @@ impl<'a> AttrMethods for JSRef<'a, Attr> { // https://dom.spec.whatwg.org/#dom-attr-prefix fn GetPrefix(self) -> Option { - self.prefix.clone() + self.prefix().as_ref().map(|p| (**p).to_owned()) } // https://dom.spec.whatwg.org/#dom-attr-ownerelement diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index 525130af9fc..8d5c2184c0f 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -1778,7 +1778,7 @@ impl Node { &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))); + attr.r().GetPrefix(), Some(copy_elem))); } }, _ => () From e8b02acb1da451bdd730fec0a98a3a1a4fc47acb Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Fri, 14 Nov 2014 16:00:23 +0100 Subject: [PATCH 2/3] Pass Atom to Attr:new for the prefix argument. --- components/script/dom/attr.rs | 6 +++--- components/script/dom/bindings/utils.rs | 5 ++--- components/script/dom/document.rs | 2 +- components/script/dom/element.rs | 13 ++++++------- components/script/dom/node.rs | 2 +- 5 files changed, 13 insertions(+), 15 deletions(-) diff --git a/components/script/dom/attr.rs b/components/script/dom/attr.rs index aef9d65e5ff..d9fba643985 100644 --- a/components/script/dom/attr.rs +++ b/components/script/dom/attr.rs @@ -109,21 +109,21 @@ pub struct Attr { impl Attr { fn new_inherited(local_name: Atom, value: AttrValue, name: Atom, namespace: Namespace, - prefix: Option, owner: Option>) -> Attr { + prefix: Option, owner: Option>) -> Attr { Attr { reflector_: Reflector::new(), local_name: local_name, value: DOMRefCell::new(value), name: name, namespace: namespace, - prefix: prefix.map(|p| Atom::from_slice(&p)), + prefix: prefix, owner: MutNullableHeap::new(owner.map(JS::from_rooted)), } } pub fn new(window: JSRef, local_name: Atom, value: AttrValue, name: Atom, namespace: Namespace, - prefix: Option, owner: Option>) -> Temporary { + prefix: Option, owner: Option>) -> Temporary { reflect_dom_object( box Attr::new_inherited(local_name, value, name, namespace, prefix, owner), GlobalRef::Window(window), diff --git a/components/script/dom/bindings/utils.rs b/components/script/dom/bindings/utils.rs index 346013f6221..9c7063d38a2 100644 --- a/components/script/dom/bindings/utils.rs +++ b/components/script/dom/bindings/utils.rs @@ -17,7 +17,6 @@ use util::str::DOMString; use libc; use libc::c_uint; -use std::borrow::ToOwned; use std::boxed; use std::cell::Cell; use std::ffi::CString; @@ -628,7 +627,7 @@ pub fn validate_qualified_name(qualified_name: &str) -> ErrorResult { /// Validate a namespace and qualified name and extract their parts. /// See https://dom.spec.whatwg.org/#validate-and-extract for details. pub fn validate_and_extract(namespace: Option, qualified_name: &str) - -> Fallible<(Namespace, Option, Atom)> { + -> Fallible<(Namespace, Option, Atom)> { // Step 1. let namespace = namespace::from_domstring(namespace); @@ -667,7 +666,7 @@ pub fn validate_and_extract(namespace: Option, qualified_name: &str) }, (ns, p) => { // Step 10. - Ok((ns, p.map(|s| s.to_owned()), Atom::from_slice(local_name))) + Ok((ns, p.map(Atom::from_slice), Atom::from_slice(local_name))) } } } diff --git a/components/script/dom/document.rs b/components/script/dom/document.rs index 8cd3655d7c5..9b88ed97a77 100644 --- a/components/script/dom/document.rs +++ b/components/script/dom/document.rs @@ -1025,7 +1025,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let (namespace, prefix, local_name) = try!(validate_and_extract(namespace, &qualified_name)); let name = QualName::new(namespace, local_name); - Ok(Element::create(name, prefix, self, ElementCreator::ScriptCreated)) + Ok(Element::create(name, prefix.map(|p| (*p).to_owned()), self, ElementCreator::ScriptCreated)) } // https://dom.spec.whatwg.org/#dom-document-createattribute diff --git a/components/script/dom/element.rs b/components/script/dom/element.rs index c17d4dce43f..b770c66caf4 100644 --- a/components/script/dom/element.rs +++ b/components/script/dom/element.rs @@ -670,12 +670,12 @@ pub trait AttributeHandlers { fn set_attribute_from_parser(self, name: QualName, value: DOMString, - prefix: Option); + prefix: Option); fn set_attribute(self, name: &Atom, value: AttrValue); fn set_custom_attribute(self, name: DOMString, value: DOMString) -> ErrorResult; fn do_set_attribute(self, local_name: Atom, value: AttrValue, name: Atom, namespace: Namespace, - prefix: Option, cb: F) + prefix: Option, cb: F) where F: Fn(JSRef) -> bool; fn parse_attribute(self, namespace: &Namespace, local_name: &Atom, value: DOMString) -> AttrValue; @@ -743,7 +743,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { fn set_attribute_from_parser(self, qname: QualName, value: DOMString, - prefix: Option) { + prefix: Option) { // 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.r().local_name() == qname.local && *a.r().namespace() == qname.ns) { @@ -753,7 +753,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { let name = match prefix { None => qname.local.clone(), Some(ref prefix) => { - let name = format!("{}:{}", *prefix, &*qname.local); + let name = format!("{}:{}", &**prefix, &*qname.local); Atom::from_slice(&name) }, }; @@ -791,7 +791,7 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> { value: AttrValue, name: Atom, namespace: Namespace, - prefix: Option, + prefix: Option, cb: F) where F: Fn(JSRef) -> bool { @@ -1099,8 +1099,7 @@ impl<'a> ElementMethods for JSRef<'a, Element> { let qualified_name = Atom::from_slice(&qualified_name); let value = self.parse_attribute(&namespace, &local_name, value); self.do_set_attribute(local_name.clone(), value, qualified_name, - namespace.clone(), prefix.map(|s| s.to_owned()), - |attr| { + namespace.clone(), prefix, |attr| { *attr.local_name() == local_name && *attr.namespace() == namespace }); diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index 8d5c2184c0f..525130af9fc 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -1778,7 +1778,7 @@ impl Node { &Attr::new(window.r(), attr.r().local_name().clone(), attr.r().value().clone(), attr.r().name().clone(), attr.r().namespace().clone(), - attr.r().GetPrefix(), Some(copy_elem))); + attr.r().prefix().clone(), Some(copy_elem))); } }, _ => () From 9435565f85e1afa8cb190a9e0c8d8bfee75ea86a Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Fri, 14 Nov 2014 16:19:24 +0100 Subject: [PATCH 3/3] Pass an Atom to Element::create. --- components/script/dom/create.rs | 8 ++++---- components/script/dom/document.rs | 2 +- components/script/dom/element.rs | 2 +- components/script/dom/node.rs | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/components/script/dom/create.rs b/components/script/dom/create.rs index 1b8e4b9f2ce..7e1f500627b 100644 --- a/components/script/dom/create.rs +++ b/components/script/dom/create.rs @@ -76,15 +76,15 @@ use dom::htmlulistelement::HTMLUListElement; use dom::htmlunknownelement::HTMLUnknownElement; use dom::htmlvideoelement::HTMLVideoElement; -use util::str::DOMString; - -use string_cache::QualName; +use string_cache::{Atom, QualName}; use std::borrow::ToOwned; -pub fn create_element(name: QualName, prefix: Option, +pub fn create_element(name: QualName, prefix: Option, document: JSRef, creator: ElementCreator) -> Temporary { + let prefix = prefix.map(|p| (*p).to_owned()); + if name.ns != ns!(HTML) { return Element::new((*name.local).to_owned(), name.ns, prefix, document); } diff --git a/components/script/dom/document.rs b/components/script/dom/document.rs index 9b88ed97a77..8cd3655d7c5 100644 --- a/components/script/dom/document.rs +++ b/components/script/dom/document.rs @@ -1025,7 +1025,7 @@ impl<'a> DocumentMethods for JSRef<'a, Document> { let (namespace, prefix, local_name) = try!(validate_and_extract(namespace, &qualified_name)); let name = QualName::new(namespace, local_name); - Ok(Element::create(name, prefix.map(|p| (*p).to_owned()), self, ElementCreator::ScriptCreated)) + Ok(Element::create(name, prefix, self, ElementCreator::ScriptCreated)) } // https://dom.spec.whatwg.org/#dom-document-createattribute diff --git a/components/script/dom/element.rs b/components/script/dom/element.rs index b770c66caf4..6aa90f0f914 100644 --- a/components/script/dom/element.rs +++ b/components/script/dom/element.rs @@ -120,7 +120,7 @@ pub enum ElementCreator { // Element methods // impl Element { - pub fn create(name: QualName, prefix: Option, + pub fn create(name: QualName, prefix: Option, document: JSRef, creator: ElementCreator) -> Temporary { create_element(name, prefix, document, creator) diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index 525130af9fc..aaea91a4b15 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -70,7 +70,7 @@ use std::iter::{FilterMap, Peekable}; use std::mem; use std::sync::Arc; use uuid; -use string_cache::QualName; +use string_cache::{Atom, QualName}; // // The basic Node structure @@ -1735,7 +1735,7 @@ impl Node { local: element.local_name().clone() }; let element = Element::create(name, - element.prefix().as_ref().map(|p| (**p).to_owned()), + element.prefix().as_ref().map(|p| Atom::from_slice(&p)), document.r(), ElementCreator::ScriptCreated); NodeCast::from_temporary(element) },