807363 - Facebook flyout document uses too much memory

Status: RESOLVED FIXED

(Firefox Graveyard :: SocialAPI: Providers, defect, P1)

Description

[:Gavin Sharp [email: [e-mail]]]

Re-posted from Asana:

Looking at the memory reports in about:memory, most frames seem to be consuming a reasonable amount of memory, except the ticker flyout. Here is the breakdown:
- content panels (jewels): 4 to 6mb each - sidebar: 9mb - worker: 16mb - ticker flyout: 41mb
It would be great if the worker could also get leaner in the longer term, but the ticker is what's most concerning at the moment. 41mb seems a lot of memory for it

Comment 1

[Florian Laplantif]

We looked at this and we weren't deleting old DOM nodes for content you had displayed a while back and that weren't relevant anymore.

We're now removing those and only keeping the 10 most recently shown divs so that should hopefully reduce memory usage a lot over time.

Comment 2

[Florian Laplantif]

(This fix will go live next week).

Comment 3

[:Gavin Sharp [email: [e-mail]]]

Awesome, thanks Florian!

Comment 4

[:Gavin Sharp [email: [e-mail]]]

So this has definitely improved - I now see reduced usage for the flyout panel (usually not more than 20MB). But I still see higher use after displaying a few entries and waiting a bit, which makes me think that maybe recently viewed content is being cached in that document longer than it needs to?