Index: /CKEditor/branches/prototype/_source/core/tools.js
===================================================================
--- /CKEditor/branches/prototype/_source/core/tools.js (revision 2149)
+++ /CKEditor/branches/prototype/_source/core/tools.js (revision 2150)
@@ -117,5 +117,5 @@
isArray : function( object )
{
- return ( object && object instanceof Array );
+ return ( !!object && object instanceof Array );
}
};
Index: /CKEditor/branches/prototype/_source/core/xml.js
===================================================================
--- /CKEditor/branches/prototype/_source/core/xml.js (revision 2149)
+++ /CKEditor/branches/prototype/_source/core/xml.js (revision 2150)
@@ -108,5 +108,5 @@
* @param {Object} [contextNode] The XML DOM node to be used as the context
* for the XPath query. The document root is used by default.
- * @returns {Array} An array containing all matched nodes. The array will
+ * @returns {ArrayLike} An array containing all matched nodes. The array will
* be empty if the query has no results.
* @example
Index: /CKEditor/branches/prototype/_source/tests/core/env.html
===================================================================
--- /CKEditor/branches/prototype/_source/tests/core/env.html (revision 2149)
+++ /CKEditor/branches/prototype/_source/tests/core/env.html (revision 2150)
@@ -29,4 +29,9 @@
assert.isFalse( CKEDITOR.env.ie );
},
+
+ test_isCompatible : function()
+ {
+ assert.isTrue( CKEDITOR.env.isCompatible );
+ },
name : document.title
Index: /CKEditor/branches/prototype/_source/tests/core/tools.html
===================================================================
--- /CKEditor/branches/prototype/_source/tests/core/tools.html (revision 2149)
+++ /CKEditor/branches/prototype/_source/tests/core/tools.html (revision 2150)
@@ -62,4 +62,24 @@
assert.areSame( fakeArray , target.prop7, 'prop7 doesn\'t match' );
},
+
+ test_isArray1 : function()
+ {
+ assert.isTrue( CKEDITOR.tools.isArray( [] ) );
+ },
+
+ test_isArray2 : function()
+ {
+ assert.isFalse( CKEDITOR.tools.isArray( { length:1 } ) );
+ },
+
+ test_isArray3 : function()
+ {
+ assert.isFalse( CKEDITOR.tools.isArray( null ) );
+ },
+
+ test_isArray4 : function()
+ {
+ assert.isFalse( CKEDITOR.tools.isArray( window.x ) );
+ },
name : document.title
Index: /CKEditor/branches/prototype/_source/tests/core/xml.html
===================================================================
--- /CKEditor/branches/prototype/_source/tests/core/xml.html (revision 2150)
+++ /CKEditor/branches/prototype/_source/tests/core/xml.html (revision 2150)
@@ -0,0 +1,142 @@
+
+
+
+ CKEDITOR.xml
+
+
+
+
+
+
+
+
Index: /CKEditor/branches/prototype/_source/tests/testall.html
===================================================================
--- /CKEditor/branches/prototype/_source/tests/testall.html (revision 2149)
+++ /CKEditor/branches/prototype/_source/tests/testall.html (revision 2150)
@@ -16,5 +16,6 @@
'core/event',
'core/scriptloader',
- 'core/tools'
+ 'core/tools',
+ 'core/xml'
];