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

Issue 165159: Merge 21550 - Disk cache: Avoid restarting the cache while it may be in use.... (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 21550 - Disk cache: Avoid restarting the cache while it may be in use. Now the code that releases resources runs from the message loop so that methods can cause the cache to disable itself while still being able to touch internal state. BUG=17604 TEST=unittests. Review URL: http://codereview.chromium.org/159327 TBR=rvargas@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22788

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -5 lines) Patch
MM net/disk_cache/backend_impl.h View 2 chunks +5 lines, -2 lines 0 comments Download
MM net/disk_cache/backend_impl.cc View 4 chunks +9 lines, -3 lines 0 comments Download
MM net/disk_cache/backend_unittest.cc View 4 chunks +5 lines, -0 lines 0 comments Download

Messages

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

Powered by Google App Engine
This is Rietveld 408576698