Disk cache: remove the hard coded list from rankings.cc...

Disk cache: remove the hard coded list from rankings.cc

The rankings module now works with any list, not just
the first one. This is part of the support needed for
new eviction algorithms.

note: Rankings::CheckList() is basically the code that
was implementing the old Rankings::SelfCheck() (moved
with this cl).

The disk cache behavior is not changing with this cl.


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

[Nicolas Sylvain, 2008-12-16 16:25:16]

lgtm

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

http://codereview.chromium.org/14141/diff/1/4#newcode648
Line 648: return ERR_INVALID_TAIL;
when head is not initialized, it's invalid tail, while when tail is not
initialized it's invalid head. Maybe a comment would be useful here.

[rvargas (doing something else), 2008-12-16 19:16:26]

Fixed,

Thanks.

On 2008/12/16 16:25:16, Nicolas Sylvain wrote:
> lgtm
> 
> http://codereview.chromium.org/14141/diff/1/4
> File net/disk_cache/rankings.cc (right):
> 
> http://codereview.chromium.org/14141/diff/1/4#newcode648
> Line 648: return ERR_INVALID_TAIL;
> when head is not initialized, it's invalid tail, while when tail is not
> initialized it's invalid head. Maybe a comment would be useful here.