New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 715859 link

Starred by 2 users
Status: Fixed
Owner:
ssid@chromium.org
Closed: Jul 2017
Cc:
█ mariakho...@chromium.org
dmu...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Reduce memory usage of Blob storage in browser for Android.

Project Member Reported by ssid@chromium.org, Apr 27 2017 
Blob storage is used by websites.
We should reduce the limit of in-memory size of blob storage on low end devices. Currently it is set to 5% of the total RAM on device. This limit will never reach before browser OOM probably.

Blob storage system should also listen to memory pressure signals. Like local storage databases, blob storage can schedule commits / eviction if we are under memory pressure.

Comment 1 by ssid@chromium.org, Apr 27 2017

Labels: Performance-Memory LowMemory
Project Member

Comment 3 by bugdroid1@chromium.org, May 12 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b26c3a43ba0a153e42639400d53ded106f5791a9

commit b26c3a43ba0a153e42639400d53ded106f5791a9
Author: ssid <ssid@chromium.org>
Date: Fri May 12 02:28:42 2017

Add memory pressure listener to Blob storage

This CL adds a memory pressure listener to blob storage memory
controller and evicts the blob items to disk on pressure. The evictions
are throttled to 30seconds to avoid thrashing disk. Records histograms
of sizes evicted with reasons.

BUG= 715859 

Review-Url: https://codereview.chromium.org/2857283002
Cr-Commit-Position: refs/heads/master@{#471189}

[modify] https://crrev.com/b26c3a43ba0a153e42639400d53ded106f5791a9/storage/browser/blob/blob_memory_controller.cc
[modify] https://crrev.com/b26c3a43ba0a153e42639400d53ded106f5791a9/storage/browser/blob/blob_memory_controller.h
[modify] https://crrev.com/b26c3a43ba0a153e42639400d53ded106f5791a9/storage/browser/blob/blob_memory_controller_unittest.cc
[modify] https://crrev.com/b26c3a43ba0a153e42639400d53ded106f5791a9/storage/common/blob_storage/blob_storage_constants.h
[modify] https://crrev.com/b26c3a43ba0a153e42639400d53ded106f5791a9/tools/metrics/histograms/histograms.xml

Comment 4 by benhenry@chromium.org, Jun 20 2017

Owner: ssid@chromium.org
Status: Assigned (was: Untriaged)
Because you're landing changes.

Comment 5 by mariakho...@chromium.org, Jun 23 2017 

Is there any work left here, or can this be closed?

Comment 6 by ssid@chromium.org, Jul 17 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment