Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-03 04:30:10 +01:00
Code Issues Projects Releases Packages Wiki Activity

Auto merge of #9282 - mattnenterprise:create-attribute-lowercase, r=nox

Browse source 
Making create attribute lowercase

This should fix #9252. I need some help getting the test case to work correctly. I get output that "TITLE" should FAIL, but it PASS. How do I make it to where it should PASS?

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9282)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2016-01-13 15:46:13 +05:30
commit ee5aead60b
2 changed files with 4 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

6
components/script/dom/document.rs
View file

@ -1847,12 +1847,14 @@ impl DocumentMethods for Document {
}
// https://dom.spec.whatwg.org/#dom-document-createattribute
fn CreateAttribute(&self, local_name: DOMString) -> Fallible<Root<Attr>> {
fn CreateAttribute(&self, mut local_name: DOMString) -> Fallible<Root<Attr>> {
if xml_name_type(&local_name) == InvalidXMLName {
debug!("Not a valid element name");
return Err(Error::InvalidCharacter);
}
if self.is_html_document {
local_name.make_ascii_lowercase();
}
let name = Atom::from(&*local_name);
// repetition used because string_cache::atom::Atom is non-copyable
let l_name = Atom::from(&*local_name);

5
tests/wpt/metadata/dom/nodes/Document-createAttribute.html.ini
View file

@ -1,5 +0,0 @@
[Document-createAttribute.html]
type: testharness
[HTML document.createAttribute("TITLE")]
expected: FAIL