622475 - [NSFW - porn site] Fennec goes into OOM state and page loading is extremely slow in image site, followup 622397

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Oleg Romashin (:romaxa)]

It is impossible to load URL, because fennec images loading is extremely slow and memory greedy.

Hope timeless can provide URL to some stress page without porn content :)

On webkit this page loaded very fast and using only ~70-80Mb vs fennec (150 and more mb)

Comment 1

[Oleg Romashin (:romaxa)]

Interesting, I changed image.mem.decodeondraw -> true, and memory consumption still huge...

Comment 2

[Oleg Romashin (:romaxa)]

With permissions.default.image = 2 (disabled images) - fennec consume for this page only ~30Mb, with disabled image placeholders pref it is ~20Mb 

Even with decodeondraw = true Fennec memory consumption on this page is huge on device it goes over 600Mb... 

Also more interesting is that Fennec process (Chrome) also consuming memory aline with content process, and at the end it drop memory back.. (~200Mb)
Need to check more why and where we allocating memory in content process

Comment 3

[Oleg Romashin (:romaxa)]

Attachment: ARM oprofile report SVG callgraph for image URL with decodeondraw = true

Interesting picture.... with decodeondraw enabled I got less CPU used by jpeg decoding and rest of CPU consumed by processing OnStatus/OnProgress messages... this looks really weird...

Comment 4

[Oleg Romashin (:romaxa)]

Attachment: Image new gfxImageSurface alloc backtrace

Backtrace for image surface allocation with decodeOnDraw enabled, joe is it expected to allocate memory which is not needed right now?

Comment 5

[Joe Drew (not getting mail)]

(In reply to comment #1)
> Interesting, I changed image.mem.decodeondraw -> true, and memory consumption
> still huge...

decode on draw only affects the behaviour of background tabs now. With decode on draw, we never decode images on background tabs (until you tab over to them); without decode on draw, we decode them as normal, but discard them if we don't tab over within a certain amount of time.

Comment 6

[Oleg Romashin (:romaxa)]

I did not get how to enable decode on draw... according to StoringSourceData, image.mem.discardable should be false and 
image.mem.decodeondraw also false

then we can get into http://mxr.mozilla.org/mozilla-central/source/modules/libpr0n/src/RasterImage.cpp#1181

Comment 7

[Martijn Wargers (dead)]

Isn't this a duplicate of bug 608385?

Comment 8

[BMO Automation]

Closing all opened bug in a graveyard component