Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fixed IE8 Memory Leak #48

Closed
wants to merge 1 commit into from

2 participants

@ronanmoynihan

...702CO.

@jrburke jrburke closed this in a1949e5
@jrburke
Owner

I did not want to go with a blanket policy of calling abort across all browsers, but instead added in the capability to regster an onXhrComplete callback in the config:
https://github.com/requirejs/text#custom-xhr-hooks

where you can do the abort call in there if you wish.

I released a new version, 2.0.6, with this change.

@ronanmoynihan

Ok, cool.

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 0 deletions.
  1. +3 −0  text.js
View
3  text.js
@@ -282,6 +282,9 @@ define(['module'], function (module) {
errback(err);
} else {
callback(xhr.responseText);
+
+ // Adding the abort call to fix a Memory Leak issue in IE8 v8.0.6001.18702CO.
+ xhr.abort();
}
}
};
Something went wrong with that request. Please try again.