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

Issue 113930: Disk cache: Reset mask_ when the disk cache is being restarted.... (Closed)

Created:
11 years, 7 months ago by rvargas (doing something else)
Modified:
9 years, 6 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Disk cache: Reset mask_ when the disk cache is being restarted. It is possible that the size for the index table is reduced when the cache is restarted (the available disk space could be lower than what it was when the cache was originally created). If that is the case, not resetting mask_ to 0 will prevent the mask to be set correctly for the new table size, resulting in a crash. BUG=12378 TEST=unittests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=17052

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+137 lines, -0 lines) Patch
A net/data/cache_tests/bad_rankings2/contents.txt View 1 chunk +66 lines, -0 lines 0 comments Download
A net/data/cache_tests/bad_rankings2/data_0 View Binary file 0 comments Download
A net/data/cache_tests/bad_rankings2/data_1 View Binary file 0 comments Download
A net/data/cache_tests/bad_rankings2/data_2 View Binary file 0 comments Download
A net/data/cache_tests/bad_rankings2/data_3 View Binary file 0 comments Download
A net/data/cache_tests/bad_rankings2/index View Binary file 0 comments Download
M net/disk_cache/backend_impl.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M net/disk_cache/backend_unittest.cc View 3 chunks +63 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
rvargas (doing something else)
11 years, 7 months ago (2009-05-28 00:32:18 UTC) #1
Nicolas Sylvain
11 years, 7 months ago (2009-05-28 00:47:59 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698