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

Issue 598834 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

[Cronet] Create a version file for disk cache

Project Member Reported by xunji...@chromium.org, Mar 29 2016

Issue description

Currently we put local_prefs.json (for HttpServerProperties) in the same directory where disk cache is. This can be problematic because simple cache backend will update the index file whenever the directory ctime changes. Since we persist HttpServerProperties to local_prefs.json every 1 min, this is costly.

Suggested approach is to put disk cache in a sub directory, and local_prefs.json in another. We also add a version file and blow out cache if the version doesn't match.
 
CL in review: https://codereview.chromium.org/1839283002/. Please feel free to leave drive-by comments. This fixes the app freeze issue that I have seen in one embedder's app. 
Status: Fixed (was: Started)

Sign in to add a comment