Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-06-24 09:04:33 +01:00
Code Issues Projects Releases Packages Wiki Activity

auto merge of #2414 : Ms2ger/servo/parser-attribute, r=SimonSapin

Browse source
This commit is contained in:
bors-servo 2014-05-13 06:55:19 -04:00
commit a9f08fd7c4
2 changed files with 24 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

14
src/components/script/dom/element.rs
View file

@ -206,7 +206,9 @@ impl<'a> ElementHelpers for JSRef<'a, Element> {
pub trait AttributeHandlers {
fn get_attribute(&self, namespace: Namespace, name: &str) -> Option<Temporary<Attr>>;
fn set_attr(&mut self, name: DOMString, value: DOMString) -> ErrorResult;
fn set_attribute_from_parser(&mut self, local_name: DOMString,
value: DOMString, namespace: Namespace,
prefix: Option<DOMString>);
fn set_attribute(&mut self, namespace: Namespace, name: DOMString,
value: DOMString) -> ErrorResult;
fn do_set_attribute(&mut self, local_name: DOMString, value: DOMString,
@ -238,8 +240,14 @@ impl<'a> AttributeHandlers for JSRef<'a, Element> {
}
}
fn set_attr(&mut self, name: DOMString, value: DOMString) -> ErrorResult {
self.set_attribute(namespace::Null, name, value)
fn set_attribute_from_parser(&mut self, local_name: DOMString,
value: DOMString, namespace: Namespace,
prefix: Option<DOMString>) {
let name = match prefix {
None => local_name.clone(),
Some(ref prefix) => format!("{:s}:{:s}", *prefix, local_name),
};
self.do_set_attribute(local_name, value, name, namespace, prefix, |_| false)
}
fn set_attribute(&mut self, namespace: Namespace, name: DOMString,

17
src/components/script/html/hubbub_html_parser.rs
View file

@ -17,7 +17,9 @@ use html::cssparse::{StylesheetProvenance, UrlProvenance, spawn_css_parser};
use script_task::Page;
use hubbub::hubbub;
use hubbub::hubbub::{NullNs, XLinkNs, XmlNs, XmlNsNs};
use servo_net::resource_task::{Load, Payload, Done, ResourceTask, load_whole_resource};
use servo_util::namespace;
use servo_util::namespace::Null;
use servo_util::str::{DOMString, HTML_SPACE_CHARACTERS};
use servo_util::task::spawn_named;
@ -327,10 +329,17 @@ pub fn parse_html(page: &Page,
debug!("-- attach attrs");
for attr in tag.attributes.iter() {
//FIXME: this should have proper error handling or explicitly drop
// exceptions on the ground
assert!(element.set_attr(attr.name.clone(),
attr.value.clone()).is_ok());
let (namespace, prefix) = match attr.ns {
NullNs => (namespace::Null, None),
XLinkNs => (namespace::XLink, Some("xlink")),
XmlNs => (namespace::XML, Some("xml")),
XmlNsNs => (namespace::XMLNS, Some("xmlns")),
ns => fail!("Not expecting namespace {:?}", ns),
};
element.set_attribute_from_parser(attr.name.clone(),
attr.value.clone(),
namespace,
prefix.map(|p| p.to_owned()));
}
//FIXME: workaround for https://github.com/mozilla/rust/issues/13246;