Bug#641541: xulrunner-6.0: eats too much (well beyond available) memory

[Mike Hommey]

On Fri, Sep 23, 2011 at 04:02:40PM +0200, Luca Capello wrote:
> clone 641541 -1
> retitle -1 conkeror: please provide an option to restore tabs on demand
> found -1 0.9.4-1
> thanks
> 
> Hi there!
> 
> On Fri, 23 Sep 2011 14:37:50 +0200, Mike Hommey wrote:
> > Please note that iceweasel does default to load all the tabs. In 8.0,
> > there is a visible pref in the "General" section, "Don't load tabs until
> > selected".
> 
> Nothing against you, but the fact that "visible" means "only when you
> choose to restore the old windows and tabs when opening" is something
> that irritates me.  Especially given that if you enable it and then
> choose again to show a blank page at opening the option stays enabled,
> which means that it is independent of the way you choose to open
> iceweasel.
> 
> Never mind, with that option enabled the situation indeed changes,
> comparing with the value reported in my previous post for 8.0:
> 
>     PID USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND
> - 31880 luca      20   0 1292m 814m  27m S   14 40.7   1:22.66 firefox-bin
> + 22761 luca      20   0  565m 133m  25m S    0  6.7   0:03.87 firefox-bin
> 
> Still, I consider 500MB for a browser a bit too much, but this is
> another problem.
> 
> > Before that, there was an about:config knob:
> > browser.sessionstore.max_concurrent_tabs
> 
> I would have thought it was browser.sessionstore.restore_hidden_tabs
> (defaults to false), which is present in 7.0~b6-1, but it has no effects
> on memory consumption.  I also tried setting the one you suggested to 1,
> again with no effect.

Sorry, I should have been more verbose, you need to set that to 0. If
you set it to 1, it will restore all tabs, one by one. The default value
is iirc 3, which means it will restore all tabs, with a maximum of three
tabs concurently being restored.

>     PID USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND
> - 13858 luca      20   0 1271m 820m  26m S   17 40.9   1:33.88 firefox-bin
> + 30985 luca      20   0 1246m 806m  26m R   16 40.2   1:33.23 firefox-bin
> 
> But given your words below, I do not care at all about old versions.
> 
> > That pref became browser.sessionstore.restore_on_demand in recent
> > versions. I don't remember in what particular version is did.
> 
> conkeror's about:config does know nothing about it (tested with
> xulrunner-8.0_8.0~a2+20110915042012-1), thus cloning and reassigning.

These prefs are browser specific, not part of xulrunner. This is
specifically something the browser has to implement itself.

Mike