Page MenuHomePhabricator

As a good platform citizen, the app should purge all cached memory objects when going to the background
Closed, ResolvedPublic3 Estimated Story Points

Description

Similar to memory warnings:

  • Article object memory
  • Images in memory
  • Section controllers in memory

Currently not valid with the latest changes:

  • WebView cache

Event Timeline

BGerstle-WMF raised the priority of this task from to Needs Triage.
BGerstle-WMF updated the task description. (Show Details)
BGerstle-WMF subscribed.

@Fjalapeno or @Mhurd [EDIT] nevermind - tracked it down

JoeWalsh subscribed.

@JoeWalsh wrote this on github too… but then I closed the pr…

Did you see what the memory savings were with this change… just curious on how much we are saving - and what the total looks like right now

So after tapping about 20 images in the Obama article and simulating a memory warning, we were stuck at 177MB before and are now at 106MB. This is simulating the worst case- entering and leaving the gallery for every image.