Ticket #1144 (new defect)

Opened 3 years ago

Last modified 10 months ago

Dojo leaks in frames in IE

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

Description (last modified by bill) (diff)

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@… 3 years ago.
Test file to reproduce the problem.
test2-frames.rar (511 bytes) - added by ornus 3 years ago.
Test file that uses baked dojo.

Change History

Changed 3 years ago by sol@…

Test file to reproduce the problem.

Changed 3 years ago by mwubben@…

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

Changed 3 years 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 3 years ago by ornus

Test file that uses baked dojo.

Changed 3 years 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 3 years ago by liucougar

  • cc sol@… added

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

Changed 3 years ago by ornus

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

Changed 3 years ago by dylan

  • milestone changed from 0.4 to 0.4.1

Changed 3 years ago by bill

  • milestone changed from 0.4.1 to 0.5

Changed 3 years ago by alex

  • milestone changed from 0.5 to 0.4.1

why was this moved?

Changed 3 years ago by ornus

just to let you know this is still an issue.

Changed 3 years 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 2 years ago by guest

Is it related to ticket #3242?

Konstantin Kolinko

Changed 2 years ago by guest

not a clue, maybe

Changed 2 years ago by alex

  • milestone changed from 0.9 to 1.0

Changed 16 months ago by bill

  • milestone changed from 1.1 to 1.2

Changed 10 months ago by bill

  • description modified (diff)
  • milestone changed from 1.2 to future
Note: See TracTickets for help on using tickets.