WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
13595
Repeatedly accessing XHR.responseText on a large file (70MB) causes memory usage to balloon
https://bugs.webkit.org/show_bug.cgi?id=13595
Summary
Repeatedly accessing XHR.responseText on a large file (70MB) causes memory us...
Antoine Quint
Reported
2007-05-05 09:58:46 PDT
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.
Attachments
Testcase
(1.13 KB, application/xhtml+xml)
2007-05-05 09:59 PDT
,
Antoine Quint
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Antoine Quint
Comment 1
2007-05-05 09:59:58 PDT
Created
attachment 14359
[details]
Testcase
Antoine Quint
Comment 2
2007-05-05 10:03:39 PDT
Comment on
attachment 14359
[details]
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.
Alexey Proskuryakov
Comment 3
2007-05-06 00:03:08 PDT
It doesn't look like there's a leak here, but garbage collection doesn't free memory quickly enough.
David Kilzer (:ddkilzer)
Comment 4
2007-07-14 16:32:21 PDT
<
rdar://problem/5335880
>
Geoffrey Garen
Comment 5
2007-07-16 15:49:51 PDT
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?
Alexey Proskuryakov
Comment 6
2007-07-25 21:01:11 PDT
Fixed by Maciej in
r24633
(as <
rdar://problem/5300291
>).
David Kilzer (:ddkilzer)
Comment 7
2007-07-25 23:41:30 PDT
(In reply to
comment #6
)
> Fixed by Maciej in
r24633
(as <
rdar://problem/5300291
>).
Follow-up fix in
r24637
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug