Index: tools/metrics/histograms/histograms.xml |
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
index 90be084e2d504169d1c419182b208044378d373e..6a55d6f8d9cb0007d75380b2892389beda676bd6 100644 |
--- a/tools/metrics/histograms/histograms.xml |
+++ b/tools/metrics/histograms/histograms.xml |
@@ -75057,6 +75057,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
<summary>The outcome of Entry::ReadData in the simple cache.</summary> |
</histogram> |
+<histogram base="true" name="SimpleCache.ReadStream1FromPrefetched" |
+ enum="Boolean"> |
+ <owner>morlovich@chromium.org</owner> |
+ <summary> |
+ Whether a read from stream 1 was satisfied from prefetch data. Reported only |
Mark P
2017/08/29 04:52:30
nit: prefetch -> prefetched
Maks Orlovich
2017/08/29 13:24:13
Done.
|
+ on first read op of entry (including if there are multiple readers, or even |
Mark P
2017/08/29 04:52:30
nit:
on first read op of entry
->
on the first rea
Maks Orlovich
2017/08/29 13:24:12
Indeed. Reads from other streams wouldn't affect t
|
+ some writers). |
Mark P
2017/08/29 04:52:30
nit: Please define / explain "stream 1". (Maybe a
Maks Orlovich
2017/08/29 13:24:13
(see below)
|
+ </summary> |
+</histogram> |
+ |
<histogram base="true" name="SimpleCache.StaleIndexExtraEntryCount" |
units="entries"> |
<owner>gavinp@chromium.org</owner> |
@@ -75139,6 +75149,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
</summary> |
</histogram> |
+<histogram base="true" name="SimpleCache.SyncOpenDidPrefetch" enum="Boolean"> |
+ <owner>morlovich@chromium.org</owner> |
+ <summary> |
+ Whether an attempt was made to prefetch the entire file when opening the |
+ entry. |
Mark P
2017/08/29 04:52:29
nit: "entry" is vague. Please be clearer. URL?
A
Maks Orlovich
2017/08/29 13:24:12
So this is obvious when one knows the disk_cache A
Mark P
2017/08/29 18:34:58
Enough so that people who don't understand the fea
Maks Orlovich
2017/08/29 19:04:10
Done, using the full C++ name to make it clear it'
Mark P
2017/08/29 19:13:26
Okay, this is good enough. However, I should ment
|
+ </summary> |
+</histogram> |
+ |
<histogram base="true" name="SimpleCache.SyncOpenEntryAge" units="hours"> |
<owner>gavinp@chromium.org</owner> |
<summary> |
@@ -92363,6 +92381,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
<affected-histogram name="SimpleCache.QueueLatency.OpenEntry"/> |
<affected-histogram name="SimpleCache.ReadIsParallelizable"/> |
<affected-histogram name="SimpleCache.ReadResult"/> |
+ <affected-histogram name="SimpleCache.ReadStream1FromPrefetched"/> |
<affected-histogram name="SimpleCache.StaleIndexExtraEntryCount"/> |
<affected-histogram name="SimpleCache.StaleIndexMissedEntryCount"/> |
<affected-histogram name="SimpleCache.StaleIndexQuality"/> |
@@ -92377,6 +92396,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
<affected-histogram name="SimpleCache.SyncCreateResult_WithIndex"/> |
<affected-histogram name="SimpleCache.SyncCreateResult_WithoutIndex"/> |
<affected-histogram name="SimpleCache.SyncKeySHA256Result"/> |
+ <affected-histogram name="SimpleCache.SyncOpenDidPrefetch"/> |
<affected-histogram name="SimpleCache.SyncOpenEntryAge"/> |
<affected-histogram name="SimpleCache.SyncOpenPlatformFileError"/> |
<affected-histogram name="SimpleCache.SyncOpenPlatformFileError_WithIndex"/> |