summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--net/disk_cache/simple/simple_index.cc15
-rw-r--r--tools/metrics/histograms/histograms.xml18
2 files changed, 28 insertions, 5 deletions
diff --git a/net/disk_cache/simple/simple_index.cc b/net/disk_cache/simple/simple_index.cc
index 8d65cbc..309614b 100644
--- a/net/disk_cache/simple/simple_index.cc
+++ b/net/disk_cache/simple/simple_index.cc
@@ -42,6 +42,8 @@ const int kDefaultWriteToDiskOnBackgroundDelayMSecs = 100;
// is left.
const uint32 kEvictionMarginDivisor = 20;
+const uint32 kBytesInKb = 1024;
+
// Utility class used for timestamp comparisons in entry metadata while sorting.
class CompareHashesForTimestamp {
typedef disk_cache::SimpleIndex SimpleIndex;
@@ -259,8 +261,10 @@ void SimpleIndex::StartEvictionIfNeeded() {
// Take all live key hashes from the index and sort them by time.
eviction_in_progress_ = true;
eviction_start_time_ = base::TimeTicks::Now();
- UMA_HISTOGRAM_COUNTS("SimpleCache.Eviction.CacheSizeOnStart", cache_size_);
- UMA_HISTOGRAM_COUNTS("SimpleCache.Eviction.MaxCacheSizeOnStart", max_size_);
+ UMA_HISTOGRAM_MEMORY_KB("SimpleCache.Eviction.CacheSizeOnStart2",
+ cache_size_ / kBytesInKb);
+ UMA_HISTOGRAM_MEMORY_KB("SimpleCache.Eviction.MaxCacheSizeOnStart2",
+ max_size_ / kBytesInKb);
scoped_ptr<std::vector<uint64> > entry_hashes(new std::vector<uint64>());
for (EntrySet::const_iterator it = entries_set_.begin(),
end = entries_set_.end(); it != end; ++it) {
@@ -288,8 +292,8 @@ void SimpleIndex::StartEvictionIfNeeded() {
UMA_HISTOGRAM_COUNTS("SimpleCache.Eviction.EntryCount", entry_hashes->size());
UMA_HISTOGRAM_TIMES("SimpleCache.Eviction.TimeToSelectEntries",
base::TimeTicks::Now() - eviction_start_time_);
- UMA_HISTOGRAM_COUNTS("SimpleCache.Eviction.SizeOfEvicted",
- evicted_so_far_size);
+ UMA_HISTOGRAM_MEMORY_KB("SimpleCache.Eviction.SizeOfEvicted2",
+ evicted_so_far_size / kBytesInKb);
index_file_->DoomEntrySet(
entry_hashes.Pass(),
@@ -316,7 +320,8 @@ void SimpleIndex::EvictionDone(int result) {
UMA_HISTOGRAM_BOOLEAN("SimpleCache.Eviction.Result", result == net::OK);
UMA_HISTOGRAM_TIMES("SimpleCache.Eviction.TimeToDone",
base::TimeTicks::Now() - eviction_start_time_);
- UMA_HISTOGRAM_COUNTS("SimpleCache.Eviction.SizeWhenDone", cache_size_);
+ UMA_HISTOGRAM_MEMORY_KB("SimpleCache.Eviction.SizeWhenDone2",
+ cache_size_ / kBytesInKb);
}
// static
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 27b364f..5aaed62 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -10729,6 +10729,10 @@ other types of suffix sets.
<summary>The size of the cache at the beginning of an eviction.</summary>
</histogram>
+<histogram name="SimpleCache.Eviction.CacheSizeOnStart2" units="KB">
+ <summary>The size of the cache at the beginning of an eviction.</summary>
+</histogram>
+
<histogram name="SimpleCache.Eviction.EntryCount">
<summary>The number of entries to be erased in an eviction.</summary>
</histogram>
@@ -10739,6 +10743,12 @@ other types of suffix sets.
</summary>
</histogram>
+<histogram name="SimpleCache.Eviction.MaxCacheSizeOnStart2" units="KB">
+ <summary>
+ The maximum allowed size of the cache at the beginning of an eviction.
+ </summary>
+</histogram>
+
<histogram name="SimpleCache.Eviction.Result" enum="BooleanSuccess">
<summary>The result of an eviction.</summary>
</histogram>
@@ -10747,10 +10757,18 @@ other types of suffix sets.
<summary>The number of bytes to be erased in an eviction.</summary>
</histogram>
+<histogram name="SimpleCache.Eviction.SizeOfEvicted2" units="KB">
+ <summary>The amount of memory freed in an eviction.</summary>
+</histogram>
+
<histogram name="SimpleCache.Eviction.SizeWhenDone" units="bytes">
<summary>The size of the cache after running an eviction.</summary>
</histogram>
+<histogram name="SimpleCache.Eviction.SizeWhenDone2" units="KB">
+ <summary>The size of the cache after running an eviction.</summary>
+</histogram>
+
<histogram name="SimpleCache.Eviction.TimeToDone" units="milliseconds">
<summary>Time spent completing an eviction.</summary>
</histogram>