Ticket #1144 (new defect)

Opened 22 months ago

Last modified 2 months ago

Dojo leaks in frames in IE

Reported by: sol@… Owned by: alex
Priority: highest Milestone: 1.2
Component: Core Version: 0.3
Severity: critical Keywords: IE frame memory+leak
Cc: sol@…

Description

Hey. It seem Dojo leaks in frames. When a page with Dojo is shown in a frame it seems dojo doesn't unload at all(?) or leaks for whatever other reason.

Simple case of just loading dojo.js with an empty page (with no other script) in a frame shows the memory leak. The leak is small (but still present) for referencing dojo in repository (not baked, when dojo.js is small), but is really a problem for built dojo (baked, dojo.js.uncompressed.js).

For release I get about 0.8MB leak for each frame on refresh. I suspect any widgets present on the page add to the leak. Perhaps unload procedure isn't run properly in the frame?

To reproduce - unpack attachment to the root of the dojo source repository (together with dojo.js file). Load test_frames.html. The problem isn't as noticeable with fresh (not built dojo) as it is with baked dojo.

I used Drip (http://outofhanwell.com/ieleak/index.php?title=Main_Page) to monitor memory usage.

Attachments

test-frames.zip (0.6 kB) - added by sol@… 22 months ago.
Test file to reproduce the problem.
test2-frames.rar (511 bytes) - added by ornus 22 months ago.
Test file that uses baked dojo.

Change History

Changed 22 months ago by sol@…

Test file to reproduce the problem.

Changed 22 months ago by mwubben@…

Could this be a case of http://novemberborn.net/javascript/edgvl ?

Changed 22 months ago by ornus

Actually, I think it could. Thanks. The test case I provided forces IE to jump in memory consumption, but it seems to stabilize after a while. I guess the problem got fixed.

Changed 22 months ago by ornus

Test file that uses baked dojo.

Changed 22 months ago by ornus

The problem is still there, especially with baked dojo (which most of the people will use). Download and unpack test2-frames to the trunk check-out folder. It assumes that built dojo is in release/dojo.

Changed 20 months ago by liucougar

  • cc sol@… added

could you confirm whether this issue is still the case with trunk?

Changed 20 months ago by ornus

Yep. It's still an issue. Baked dojo leaks much more than non-baked, but the leak is still there.

Changed 19 months ago by dylan

  • milestone changed from 0.4 to 0.4.1

Changed 19 months ago by bill

  • milestone changed from 0.4.1 to 0.5

Changed 18 months ago by alex

  • milestone changed from 0.5 to 0.4.1

why was this moved?

Changed 18 months ago by ornus

just to let you know this is still an issue.

Changed 18 months ago by bill

  • milestone changed from 0.4.1 to 0.5

Pushing to 0.5 since we ran out of time on 0.4.1.

Changed 12 months ago by guest

Is it related to ticket #3242?

Konstantin Kolinko

Changed 12 months ago by guest

not a clue, maybe

Changed 10 months ago by alex

  • milestone changed from 0.9 to 1.0

Changed 2 months ago by bill

  • milestone changed from 1.1 to 1.2
Note: See TracTickets for help on using tickets.