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

Implement removeAttributeNode

Browse source 
This commit implement removeAttributeNode, as described here:
https://dom.spec.whatwg.org/#dom-element-removeattributenode

See also #8724 and #8068.
This commit is contained in:
Nick Fitzgerald 2015-11-28 16:27:51 -08:00 • committed by Anthony Ramine
parent 918681931f
commit d28c9a1588
5 changed files with 8 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -1358,6 +1358,12 @@ impl ElementMethods for Element {
self.remove_attribute(&namespace, &local_name);
}
// https://dom.spec.whatwg.org/#dom-element-removeattributenode
fn RemoveAttributeNode(&self, attr: &Attr) -> Fallible<Root<Attr>> {
self.remove_first_matching_attribute(|a| a == attr)
.ok_or(Error::NotFound)
}
// https://dom.spec.whatwg.org/#dom-element-hasattribute
fn HasAttribute(&self, name: DOMString) -> bool {
self.GetAttribute(name).is_some()

2
components/script/dom/webidls/Element.webidl
View file

@ -49,6 +49,8 @@ interface Element : Node {
void setAttributeNS(DOMString? namespace, DOMString name, DOMString value);
void removeAttribute(DOMString name);
void removeAttributeNS(DOMString? namespace, DOMString localName);
[Throws]
Attr removeAttributeNode(Attr oldAttr);
boolean hasAttribute(DOMString name);
boolean hasAttributeNS(DOMString? namespace, DOMString localName);

9
tests/wpt/metadata/dom/interfaces.html.ini
View file

@ -108,9 +108,6 @@
[Element interface: operation hasAttributes()]
expected: FAIL
[Element interface: operation removeAttributeNode(Attr)]
expected: FAIL
[Element interface: operation query(DOMString)]
expected: FAIL
@ -120,12 +117,6 @@
[Element interface: element must inherit property "hasAttributes" with the proper type (7)]
expected: FAIL
[Element interface: element must inherit property "removeAttributeNode" with the proper type (22)]
expected: FAIL
[Element interface: calling removeAttributeNode(Attr) on element with too few arguments must throw TypeError]
expected: FAIL
[Element interface: element must inherit property "query" with the proper type (34)]
expected: FAIL

3
tests/wpt/metadata/dom/nodes/attributes.html.ini
View file

@ -1,8 +1,5 @@
[attributes.html]
type: testharness
[Basic functionality of removeAttributeNode]
expected: FAIL
[Own property names should only include all-lowercase qualified names for an HTML element in an HTML document]
expected: FAIL

6
tests/wpt/metadata/html/dom/interfaces.html.ini
View file

@ -1986,12 +1986,6 @@
[Element interface: document.createElement("noscript") must inherit property "hasAttributes" with the proper type (7)]
expected: FAIL
[Element interface: document.createElement("noscript") must inherit property "removeAttributeNode" with the proper type (22)]
expected: FAIL
[Element interface: calling removeAttributeNode(Attr) on document.createElement("noscript") with too few arguments must throw TypeError]
expected: FAIL
[Element interface: document.createElement("noscript") must inherit property "query" with the proper type (34)]
expected: FAIL