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

auto merge of #1710 : jdm/servo/idmap-in-doc, r=Ms2ger

Browse source 
...ent.
This commit is contained in:
bors-servo 2014-02-18 15:23:11 -05:00
commit 01fcf3c877
2 changed files with 11 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -223,7 +223,7 @@ impl Element {
"style" => {
self.style_attribute = Some(style::parse_style_attribute(value))
}
"id" => {
"id" if abstract_self.is_in_doc() => {
// XXX: this dual declaration are workaround to avoid the compile error:
// "borrowed value does not live long enough"
let doc = self.node.owner_doc();
@ -287,7 +287,7 @@ impl Element {
"style" => {
self.style_attribute = None
}
"id" => {
"id" if abstract_self.is_in_doc() => {
// XXX: this dual declaration are workaround to avoid the compile error:
// "borrowed value does not live long enough"
let doc = self.node.owner_doc();

9
src/test/html/content/test_document_getElementById.html
View file

@ -66,6 +66,15 @@
is(e2, null, "test3-2, the method should return null when the passed id is none in document.");
}
{
// Ensure that the id attribute only affects elements present in a document
let e = document.createElement('div');
e.id = "should-not-exist";
is(document.getElementById("should-not-exist"), null);
document.body.appendChild(e);
is(document.getElementById("should-not-exist"), e);
}
// TODO:
// test4: "in tree order, within the context object's tree"
// http://dom.spec.whatwg.org/#dom-document-getelementbyid.