Chromium Code Reviews
Help | Chromium Project | Sign in
(496)

Issue 155314: Disk cache: Don't evict entries if we are busy doing other stuff.... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 9 months ago by rvargas (OOO until 4_21)
Modified:
2 years, 10 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews_googlegroups.com, darin, willchan
Visibility:
Public.

Description

Disk cache: Don't evict entries if we are busy doing other stuff.

BUG=10727
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20330

Patch Set 1 #

Patch Set 2 : '' #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+58 lines, -20 lines) Lint Patch
M net/disk_cache/backend_impl.h View 1 chunk +6 lines, -3 lines 0 comments 0 errors Download
M net/disk_cache/backend_impl.cc View 1 2 chunks +20 lines, -7 lines 1 comment 0 errors Download
M net/disk_cache/eviction.h View 2 chunks +3 lines, -0 lines 0 comments 0 errors Download
M net/disk_cache/eviction.cc View 8 chunks +29 lines, -10 lines 0 comments 0 errors Download
Commit:

Messages

Total messages: 4
rvargas (OOO until 4_21)
4 years, 9 months ago #1
Nicolas Sylvain
LGTM
4 years, 9 months ago #2
rvargas (OOO until 4_21)
I added the extra histogram. On 2009/07/09 21:51:10, Nicolas Sylvain wrote: > LGTM
4 years, 9 months ago #3
Nicolas Sylvain
4 years, 9 months ago #4
http://codereview.chromium.org/155314/diff/9/1010
File net/disk_cache/backend_impl.cc (right):

http://codereview.chromium.org/155314/diff/9/1010#newcode881
Line 881: CACHE_UMA(COUNTS, "NumberOfReferences", 0, num_refs_);
still lgtm
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6