Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 204910 - IE7 leaks memory on each request
Summary: IE7 leaks memory on each request
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.0   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 1.0 Final   Edit
Assignee: Rüdiger Herrmann CLA
QA Contact:
URL:
Whiteboard: qx-closed
Keywords:
Depends on:
Blocks:
 
Reported: 2007-09-28 10:16 EDT by Rüdiger Herrmann CLA
Modified: 2008-09-24 05:51 EDT (History)
1 user (show)

See Also:


Attachments
Fixes the problem by using a dummy function (1.35 KB, patch)
2007-09-29 05:18 EDT, Hugh Gibson CLA
jkrause: iplog+
Details | Diff
Possible more extensive fix removing timed call to handler (2.52 KB, patch)
2007-09-29 05:20 EDT, Hugh Gibson CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rüdiger Herrmann CLA 2007-09-28 10:16:18 EDT
There is a bug report in qooxdoo that describes the problem in depth:
  http://bugzilla.qooxdoo.org/show_bug.cgi?id=664
  
Opened this bug to keep track of the progress and have a place to possibly attach a patch.
Comment 1 Hugh Gibson CLA 2007-09-29 05:18:30 EDT
Created attachment 79436 [details]
Fixes the problem by using a dummy function

This is the basic fix that should work in all circumstances.

It's possible that a better fix can be used but I haven't been able to test it. It relates to http://bugzilla.qooxdoo.org/show_bug.cgi?id=190.

Hugh Gibson
Comment 2 Hugh Gibson CLA 2007-09-29 05:20:37 EDT
Created attachment 79437 [details]
Possible more extensive fix removing timed call to handler

The previous patch may also fix the problem in http://bugzilla.qooxdoo.org/show_bug.cgi?id=190 but I haven't been able to test it.

If so, the hack applied in the original fix for qooxdoo issue 190 (to call onreadystatechange on a timer) can be removed. This patch does that.

Someone who had the original issue 190 problem should test this patch thoroughly before it is applied.

Hugh Gibson
Comment 3 Rüdiger Herrmann CLA 2007-10-01 15:15:11 EDT
Hugh, thank you very much for the patches. This helped a lot!
With the 1.0 release that close, I decided to apply the first patch.
Changes committed to CVS HEAD.