19187 - [dogfood]Memory leaks leading to crash with dsMemFullErr.

Status: VERIFIED FIXED

(Core :: Layout, defect, P3)

Description

[Steve Dagley]

Loading http://www.stomped.com/articles/q3acr1.html seems to consume lots of
memory and crash consistently for me.  If you don't crash on the first load try
loading one of the character renderings from the links in the article then going
back.

Comment 1

[Simon Fraser [no longer active]]

This is GWorld badness. Image leaks (bugs 15818, 15820, 15821) are part of the
raeson, but we should still not crash.

Comment 2

[Simon Fraser [no longer active]]

Making dogfood. This bug causes the browser to be unusable for more than a few
minutes on Mac.

Comment 3

[leger]

Putting on PDT+ radar.

Comment 4

[Simon Fraser [no longer active]]

This will have been fixed by the fixes for 19114. I verified using ZoneRanger,
and the debugger, that we are actually freeing image data now.

Comment 5

[Christine Hoffman]

[e-mail]:

I need to get this verified today if possible. I do not have sufficient info to
tract leaks. Can you take a look at this and, if it is fixed, verify it so?
Thank.

Comment 6

[Steve Dagley]

If sfraser sez he verified it that's good enough for me.  Of course QA is the
ultimate authority on marking things VERIFIED so I suspect they'll test it
themselves and they can ask me if they have any questions on the repro steps.

Comment 7

[Simon Fraser [no longer active]]

pink says he's still getting crashes, so I'm reopening.

Comment 8

[rubydoo123]

clearing reopen and assigning

Comment 9

[Simon Fraser [no longer active]]

I'm gonna take this off the PDT list, since a real solution requires some low-
memory handling work (bug 20743).

Comment 10

[leger]

How frequent are the crashes?  Less frequent PDT-, more frequent...impacting
daily work PDT+...fill us in please.  Is there a workaround by increasing the
partition size?

Comment 11

[Simon Fraser [no longer active]]

Yes, increasing the partition size, so that the amount of free memory just after
startup is > 2Mb, should work. Only some people have reported crashes
(pinkerton).

Comment 12

[leger]

Cool - Putting on PDT+ radar.  Let's crank up the partition for dogfood. Moving
to M12 per chofmann.

Comment 13

[Simon Fraser [no longer active]]

This bug has been fixed by kmcclusk's change to fix bug 19114 -- reducing the
imglib cache from 2 to 1Mb. I don't need to do anything.

Comment 14

[Chris Petersen]

Marking verified per last comments.

Comment 15

[aleth [:aleth]]

Attachment: Port Bug 1244743 - Replace MOZ_NATIVE_X with MOZ_SYSTEM_X

Comment 16

[aleth [:aleth]]

Oh dear, bzexport messed up and picked the wrong bug number.