Let NamedNodeMap take and return Attr instead of Node

LayoutTests/dom/xhtml/level2/core/hc_namednodemapinvalidtype1.js

Index: LayoutTests/dom/xhtml/level2/core/hc_namednodemapinvalidtype1.js
diff --git a/LayoutTests/dom/xhtml/level2/core/hc_namednodemapinvalidtype1.js b/LayoutTests/dom/xhtml/level2/core/hc_namednodemapinvalidtype1.js
index 0cffa0e185bed79bc8a7061c33008cf1406988f3..ec4db837a6730ea252d395a9ea3875fd2ecb3701 100644
--- a/LayoutTests/dom/xhtml/level2/core/hc_namednodemapinvalidtype1.js
+++ b/LayoutTests/dom/xhtml/level2/core/hc_namednodemapinvalidtype1.js
@@ -109,9 +109,9 @@ function hc_namednodemapinvalidtype1() {
             retval = attributes.setNamedItem(newElem);
         }
 		catch(ex) {
-      success = (typeof(ex.code) != 'undefined' && ex.code == 3);
+      success = ex.name == 'TypeError';
 		}
-		assertTrue("throw_HIERARCHY_REQUEST_ERR",success);
+		assertTrue("throw_TYPE_ERR",success);

[Mike West, 2014/11/25 14:04:01]

Tabs? :(

[philipj_slow, 2014/11/25 14:46:53]

Yeah, these crufty old tests are a crazy mix of tabs and spaces. No tab width
makes it look sane, so I try to leave it alone. Would you like to review a
mass-cleanup CL? ;)

 	}
 
 }