From 63bfe1ded1fe01d326594e9a757a2c2ceab8d8ae Mon Sep 17 00:00:00 2001 From: Ariel Flesler Date: Sat, 18 Oct 2008 22:22:08 +0000 Subject: [PATCH] jquery core: closes #3485. Fixing a memory leak on IE. --- src/core.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/core.js b/src/core.js index 829e153..9279d17 100644 --- a/src/core.js +++ b/src/core.js @@ -612,10 +612,14 @@ jQuery.extend({ }, // See test/unit/core.js for details concerning this function. - // Since 1.3 DOM methods and function like alert + // Since version 1.3, DOM methods and functions like alert // aren't supported. They return false on IE (#2968). + + // Memory leaks appear in IE6 when applying instanceof + // to the window, document or any other COM object (#3485) + // http://ajaxian.com/archives/working-aroung-the-instanceof-memory-leak isFunction: function( fn ) { - return fn instanceof Function; + return !!fn && !!fn.hasOwnProperty && fn instanceof Function; }, // check if an element is in a (or is an) XML document -- 1.7.10.4