My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 98238: Remove MemoryPurger
7 people starred this issue and may be notified of changes. Back to list
 
Project Member Reported by pkasting@chromium.org, Sep 27, 2011
People think it should be used in memory pressure cases when it shouldn't.

Anyone using this for the above reason should immediately be changed not to use this class.
Sep 27, 2011
#1 tonyg@chromium.org
Is this a good example of a potential problem with it: https://bugs.webkit.org/show_bug.cgi?id=67031

Could you add any more details?
Cc: satish@chromium.org mnaganov@chromium.org tonyg@chromium.org
Sep 27, 2011
#2 pkasting@chromium.org
The problem is that it is not designed to reduce memory pressure, it is designed to flush literally everything possible in preparation for suspend-to-disk.  Accordingly almost everything it purges is something that needs to be recreated immediately in normal use, and so rather than reducing memory pressure it just causes thrashing and poor UX.  Someone responding to a low memory notification might need to do a few of the things the purger does, but mostly will want to go in completely different directions, like trying to shut down long-unused tabs.

The bug you link is not a reason why I want to remove the memory purger.  It _is_ a potential problem, which should be solved by someone who understands the semantics of the font cache.
Oct 5, 2011
#3 pkasting@chromium.org
CCing pinkerton.  Mike, you made an argument in a recent email thread that we should keep this class around, and use it not for low memory notifications but instead for backgrounding the app in low-memory environments (which may make Chrome take longer to be re-foregrounded, but perhaps that tradeoff is fine).  If you're committed to that, we could perhaps close this bug, but I'd be looking for someone to take ownership of this code so it's not dead and it's known to be in real, useful use by someone.  At that point we might want to modify it to e.g. drop the backing store for the foreground tab (we might already drop the other backing stores, or maybe we don't?).
Cc: pinkerton@chromium.org
Nov 2, 2012
#4 satish@chromium.org
(No comment was entered for this change.)
Cc: -satish@chromium.org bul...@chromium.org
Mar 10, 2013
#5 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-Internals Cr-Internals
May 15, 2013
#6 tfar...@chromium.org
(No comment was entered for this change.)
Labels: -Pri-2 Pri-3
May 29, 2013
#7 bul...@chromium.org
(No comment was entered for this change.)
Cc: pliard@chromium.org
May 30, 2013
#8 rmcil...@chromium.org
(No comment was entered for this change.)
Cc: rmcil...@chromium.org
Apr 1, 2014
#9 pkasting@chromium.org
I killed MemoryPurger a month or so ago on a different bug.
Status: WontFix
Sign in to add a comment

Powered by Google Project Hosting