13595 2007-05-05 09:58:46 -0700 Repeatedly accessing XHR.responseText on a large file (70MB) causes memory usage to balloon 2007-07-25 23:41:30 -0700 1 1 1 Unclassified WebKit XML 523.x (Safari 3) Mac (Intel) OS X 10.4 RESOLVED FIXED InRadar P2 Major --- 1 ml webkit-unassigned ap mjs mrowe oldest_to_newest 10893 0 ml 2007-05-05 09:58:46 -0700 Loading a large video file using XMLHttpRequest make the nightly WebKit application memory shoot up to about 800MB in 5 seconds, and keeps going up until no memory is left. 10895 1 14359 ml 2007-05-05 09:59:58 -0700 Created attachment 14359 Testcase 10896 2 14359 ml 2007-05-05 10:03:39 -0700 Comment on attachment 14359 Testcase This testcase needs to be opened from the local filesystem given that it makes a call to another domain and would result in a JS exception if opened directly from the link within the bug. 10932 3 ap 2007-05-06 00:03:08 -0700 It doesn't look like there's a leak here, but garbage collection doesn't free memory quickly enough. 4714 4 ddkilzer 2007-07-14 16:32:21 -0700 <rdar://problem/5335880> 4591 5 ggaren 2007-07-16 15:49:51 -0700 In particular, it looks like the problem is caused by this line var currentLength = xhr.responseText.length; which causes the network data to be serialized to text every time a new chunk arrives. Is there a real website that uses this technique? 3703 6 ap 2007-07-25 21:01:11 -0700 Fixed by Maciej in r24633 (as <rdar://problem/5300291>). 3692 7 ddkilzer 2007-07-25 23:41:30 -0700 (In reply to comment #6) > Fixed by Maciej in r24633 (as <rdar://problem/5300291>). Follow-up fix in r24637. 14359 2007-05-05 09:59:58 -0700 2007-05-05 09:59:58 -0700 Testcase massive_leak.xml application/xhtml+xml 1160 ml PGh0bWwgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGh0bWwiPgogIAogIDxoZWFkPgog ICAgPHRpdGxlPk1hc3NpdmUgTWVtb3J5IExlYWs8L3RpdGxlPgogICAgPHNjcmlwdD4KICAgICAg PCFbQ0RBVEFbCgogICAgICAgIGZ1bmN0aW9uIGxvYWRWaWRlbyAodmlkZW9VUkkpIHsKICAgICAg ICAgIHZhciB4aHIgPSBuZXcgWE1MSHR0cFJlcXVlc3QoKTsKICAgICAgICAgIHhoci5vcGVuKCdH RVQnLCB2aWRlb1VSSSwgdHJ1ZSk7CiAgICAgICAgICB4aHIub25yZWFkeXN0YXRlY2hhbmdlID0g ZnVuY3Rpb24gKHhockV2ZW50KSB7CiAgICAgICAgICAgIGlmICh4aHIucmVhZHlTdGF0ZSA9PSAz KSB7CiAgICAgICAgICAgICAgdmFyIGN1cnJlbnRMZW5ndGggPSB4aHIucmVzcG9uc2VUZXh0Lmxl bmd0aDsKICAgICAgICAgICAgICB2YXIgdG90YWxMZW5ndGggPSBwYXJzZUludCh4aHIuZ2V0UmVz cG9uc2VIZWFkZXIoJ0NvbnRlbnQtTGVuZ3RoJyksIDEwKTsKICAgICAgICAgICAgICBwcmludCgo TWF0aC5yb3VuZCgoY3VycmVudExlbmd0aCAvIHRvdGFsTGVuZ3RoKSAqIDEwMDAwKSAvIDEwMCkg KyAnJScpOwogICAgICAgICAgICB9CiAgICAgICAgICAgIGVsc2UgaWYgKHhoci5yZWFkeVN0YXRl ID09IDQpIHsKICAgICAgICAgICAgICBpZiAoeGhyLnN0YXR1cyA9PSAyMDApIHsKICAgICAgICAg ICAgICAgIGFsZXJ0KCdkb25lIGxvYWRpbmcnKTsKICAgICAgICAgICAgICB9CiAgICAgICAgICAg ICAgZWxzZSB7CiAgICAgICAgICAgICAgICBhbGVydCgnTk9LJyk7CiAgICAgICAgICAgICAgfQog ICAgICAgICAgICB9CiAgICAgICAgICB9OwogICAgICAgICAgeGhyLnNlbmQobnVsbCk7CiAgICAg ICAgfQoKICAgICAgICBmdW5jdGlvbiBwcmludCAobWVzc2FnZSkgewogICAgICAgICAgZG9jdW1l bnQuZ2V0RWxlbWVudEJ5SWQoJ291dGxldCcpLnRleHRDb250ZW50ID0gbWVzc2FnZTsKICAgICAg ICB9CgogICAgICAgIGxvYWRWaWRlbygnaHR0cDovL3N0cmVhbW9zLmF0bHJlYy5jb20vZG93bmxv YWQvYXRsYW50aWMvYmpvcmsvdmlkZW8vYmpvcmtfaXR1bmVzMS5tNHYnKTsKCiAgICAgIF1dPgog ICAgPC9zY3JpcHQ+CiAgPC9oZWFkPgogIAogIDxib2R5PgogICAgPHAgaWQ9Im91dGxldCIgLz4K ICA8L2JvZHk+CiAgCjwvaHRtbD4=