Fix for memory leak in SC.Response

aflatter · Sep 8, 2010 · e2474b7 · e2474b7
1 parent 9d3b2ce
commit e2474b7
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/frameworks/foundation/system/response.js b/frameworks/foundation/system/response.js
@@ -529,8 +529,14 @@ SC.XHRResponse = SC.Response.extend({
         this.set('status', status);
 
       }, this);
-      // avoid memory leak in MSIE: clean up
-      rawRequest.onreadystatechange = function() {} ;
+
+      // Avoid memory leaks
+      if (!SC.browser.msie) {
+        SC.Event.remove(rawRequest, 'readystatechange', this, this.finishRequest);	  
+      } else {
+        rawRequest.onreadystatechange = null;
+      }
+
       return YES;
     }
     return NO;