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

Issue 8065015: Disk Cache: Improve handling of dirty entries. (Closed)

Created:
9 years, 2 months ago by rvargas (doing something else)
Modified:
9 years, 2 months ago
Reviewers:
gavinp
CC:
chromium-reviews, cbentzel+watch_chromium.org, darin-cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Disk Cache: Improve handling of dirty entries. * Split the entry sanity checks in two parts: a critical one and a non-critical one. This allows us to return dirty entries instead of failing to open them. * Make sure that we cannot reach an entry through the index before the actual data reaches the disk (when creating an entry linked through a parent we were not respecting that). * When deleting a block from a block file, first clean it up and then update the map (avoid leaving a dirty free block if there is a crash) * Handle the case of errors when opening entries through the enumerations path. BUG=73102 TEST=net_unittests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=103323

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+528 lines, -62 lines) Patch
M net/disk_cache/backend_impl.h View 1 chunk +2 lines, -2 lines 0 comments Download
M net/disk_cache/backend_impl.cc View 6 chunks +51 lines, -14 lines 1 comment Download
M net/disk_cache/backend_unittest.cc View 3 chunks +340 lines, -0 lines 0 comments Download
M net/disk_cache/block_files.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M net/disk_cache/entry_impl.h View 1 chunk +8 lines, -0 lines 0 comments Download
M net/disk_cache/entry_impl.cc View 6 chunks +51 lines, -8 lines 0 comments Download
M net/disk_cache/eviction.h View 1 chunk +1 line, -1 line 0 comments Download
M net/disk_cache/eviction.cc View 12 chunks +31 lines, -16 lines 0 comments Download
M net/disk_cache/in_flight_backend_io.h View 1 chunk +3 lines, -1 line 0 comments Download
M net/disk_cache/rankings.h View 3 chunks +13 lines, -5 lines 0 comments Download
M net/disk_cache/rankings.cc View 6 chunks +25 lines, -12 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
rvargas (doing something else)
9 years, 2 months ago (2011-09-28 18:43:14 UTC) #1
gavinp
9 years, 2 months ago (2011-09-29 13:25:34 UTC) #2
LGTM, with the typo fixed.

http://codereview.chromium.org/8065015/diff/1/net/disk_cache/backend_impl.cc
File net/disk_cache/backend_impl.cc (right):

http://codereview.chromium.org/8065015/diff/1/net/disk_cache/backend_impl.cc#...
net/disk_cache/backend_impl.cc:765: // and finally thorugh the lists. If there
is a crash in this process, we may
typo: thorugh -> through

http://codereview.chromium.org/8065015/diff/1/net/disk_cache/rankings.cc
File net/disk_cache/rankings.cc (right):

http://codereview.chromium.org/8065015/diff/1/net/disk_cache/rankings.cc#newc...
net/disk_cache/rankings.cc:819: bool Rankings::IsHead(CacheAddr addr, List*
list) const {
Good catch.

Powered by Google App Engine
This is Rietveld 408576698