Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1242)

Issue 164187: Merge 21762 - Disk Cache: Delete chained block files when they become empty.... (Closed)

Created:
11 years, 4 months ago by laforge
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com, darin (slow to review), willchan no longer on Chromium
Visibility:
Public.

Description

Merge 21762 - Disk Cache: Delete chained block files when they become empty. We were leaving empty block files in the chain, and worst of all, not reusing them because we were thinking that these files were "almost full". Now we also check for empty files when the cache starts. BUG=16740 TEST=unittest. Review URL: http://codereview.chromium.org/159451 TBR=rvargas@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22789

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+121 lines, -22 lines) Patch
MM net/disk_cache/block_files.h View 1 chunk +3 lines, -0 lines 0 comments Download
MM net/disk_cache/block_files.cc View 6 chunks +62 lines, -16 lines 0 comments Download
MM net/disk_cache/block_files_unittest.cc View 2 chunks +53 lines, -3 lines 0 comments Download
MM net/disk_cache/entry_impl.cc View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
laforge
11 years, 4 months ago (2009-08-07 21:13:53 UTC) #1
rvargas (doing something else)
11 years, 4 months ago (2009-08-07 21:18:12 UTC) #2
OK

Powered by Google App Engine
This is Rietveld 408576698