Ajax: better responseText copy logic
Reported by Mark Caudill | July 9th, 2009 @ 11:52 PM
Ensure responseText is only copied if it will be used in onInteractive or the readyState is 4 (complete). This keeps aborted/timedout request's footprint down.
Comments and changes to this ticket
-
Mark Caudill July 10th, 2009 @ 01:26 PM
- Tag changed from ajax to ajax, patched
-
John-David Dalton August 3rd, 2009 @ 02:43 AM
Resolved by https://github.com/jdalton/fusejs/commit/6f46946ada5bbe34bc78015bd90e9a0d0175d6b7
By ensuring each state is called only 1 time and only setting responseText in 3 and 4 we reduce the memory issue.
We still need to evaluate it because of global responders that could be attached.
checking those and the request callacks might be an option if the problem persists after this patch. -
John-David Dalton August 3rd, 2009 @ 02:50 AM
- State changed from new to resolved
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
JavaScript frameworks share similar features and functionality such as DOM manipulation, event registration, and CSS selector engines. FuseJS attempts to incorporate the strengths of these frameworks into one stable, efficient, and optimized core JavaScript framework.