evict larger entries first

net/disk_cache/simple/simple_experiment.cc

Index: net/disk_cache/simple/simple_experiment.cc
diff --git a/net/disk_cache/simple/simple_experiment.cc b/net/disk_cache/simple/simple_experiment.cc
index 38ea589322f97df405f9b48bcaa349510847ce52..df00bb06964efb04ee28fb9340fe2f7907067d16 100644
--- a/net/disk_cache/simple/simple_experiment.cc
+++ b/net/disk_cache/simple/simple_experiment.cc
@@ -15,6 +15,9 @@ namespace disk_cache {
 
 const base::Feature kSimpleSizeExperiment = {"SimpleSizeExperiment",

[pasko, 2017/06/23 15:10:30]

maybe allow a param like log/sqrt/whatever that you initially considered? Given
that we are running an experiment anyway, it would be a cheap way to discover
whether typical uses are matching the data you experimented with locally.

I'm OK with doing it in a later change.

[hubbe, 2017/06/23 18:25:52]

log & sqrt are too slow, don't make mathematical sense and they don't performed
well in my experiments so far. I'm not going to try that unless there is some
reason suggesting that it would be better.

                                              base::FEATURE_DISABLED_BY_DEFAULT};
+const base::Feature kSimpleCacheEvictionWithSize = {
+    "SimpleCacheEvictionWithSize", base::FEATURE_DISABLED_BY_DEFAULT};

[pasko, 2017/06/23 15:10:30]

does finch config allow enabling more than one feature and providing params for
each one of them?

I think in order for this to allow dropping caches in a no-op group, we would
either need to enable SimpleSizeExperiment or use the similar logic as
CheckForSimpleSizeExperiment/SimpleExperimentMatches.

[hubbe, 2017/06/23 18:25:52]

I don't intend to drop caches in the finch experiment.
If I did, I would use the SimpleExperiment logic in this file to do that.

[pasko, 2017/06/26 13:32:13]

Thank you. I have a few reservations about it, so I started a discussion in the
document.

not lgtm until we reach agreement in the discussion, feel free to throw a VC to
figure out what works for us.

+
 const char kSizeMultiplierParam[] = "SizeMultiplier";
 
 namespace {