Ductyping galore, and do less work in all identify functions

Author: Munter

lib/index.js

@@ -1,4 +1,3 @@
-/*global HTMLDocument*/
 function getAttributes(elm) {
   var attrs = elm.attributes;
   var result = {};
@@ -77,11 +76,7 @@ module.exports = {
       name: 'DOMNode',
       base: 'object',
       identify: function (obj) {
-        if (!obj) {
-          return false;
-        }
-
-        return obj.nodeName && [2, 3, 4, 5, 6, 7, 9, 10, 11, 12].indexOf(obj.nodeType) > -1;
+        return obj && obj.nodeName && [2, 3, 4, 5, 6, 7, 10, 11, 12].indexOf(obj.nodeType) > -1;
       },
       equal: function (a, b) {
         return a.nodeValue === b.nodeValue;
@@ -95,11 +90,7 @@ module.exports = {
       name: 'DOMComment',
       base: 'DOMNode',
       identify: function (obj) {
-        if (!obj) {
-          return false;
-        }
-
-        return obj.nodeType === 8;
+        return obj && obj.nodeType === 8;
       },
       equal: function (a, b) {
         return a.nodeValue === b.nodeValue;
@@ -113,28 +104,15 @@ module.exports = {
       name: 'HTMLDocument',
       base: 'DOMNode',
       identify: function (obj) {
-        if ('HTMLDocument' in this) {
-          return obj instanceof HTMLDocument;
-        }
-
-        if (typeof window !== 'undefined') {
-          return obj instanceof window.HTMLDocument;
-        }
-
-        // Stupid duck typing case. Help :)
-        return obj && obj.documentElement && obj.implementation;
+        return obj && obj.nodeType === 9 && obj.documentElement && obj.implementation;
       }
     });
 
     expect.addType({
       name: 'HTMLElement',
       base: 'DOMNode',
       identify: function (obj) {
-        if (!obj) {
-          return false;
-        }
-
-        return !!(obj.nodeType === 1 && obj.nodeName && obj.attributes && obj.outerHTML);
+        return obj && obj.nodeType === 1 && obj.nodeName && obj.attributes && obj.outerHTML;
       },
       equal: function (a, b) {
         return stringifyElement(a) === stringifyElement(b);