Index: tools/metrics/histograms/histograms.xml |
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
index 838f9a1a91e81d15901f82eda0d30fb439f55296..2e430c1555bb2d2bb1c3d4d5e42bded9fb9460bd 100644 |
--- a/tools/metrics/histograms/histograms.xml |
+++ b/tools/metrics/histograms/histograms.xml |
@@ -937,6 +937,72 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
</summary> |
</histogram> |
+<histogram name="Android.PhotoPicker.BitmapScalerTask" units="ms"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary> |
+ Measures the amount of time the BitmapScaler spends scaling per bitmap. |
Mark P
2017/06/02 17:17:58
nit: per -> a
nit: spends -> spent
Finnur
2017/06/06 14:23:16
Done.
|
+ </summary> |
+</histogram> |
+ |
+<histogram name="Android.PhotoPicker.CacheHits" units="Hits"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary> |
+ Measures how often an image is served from the high-res cache while the |
Mark P
2017/06/02 17:17:58
see similar comments to DecodeRequests, as my same
Finnur
2017/06/06 14:23:16
No problem. Done (see answer below).
|
+ dialog is open. |
+ </summary> |
+</histogram> |
+ |
+<histogram name="Android.PhotoPicker.DecodeRequests" units="Hits"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary> |
+ Measures how often an image is sent for decoding while the dialog is open. |
Mark P
2017/06/02 17:17:58
nit: "how often" sounds like a rate. You're simpl
Finnur
2017/06/06 14:23:16
Yeah, it's not a rate and it's not just counted wh
Mark P
2017/06/07 17:19:30
The revised comment reads great now! thanks!
|
+ </summary> |
+</histogram> |
+ |
+<histogram name="Android.PhotoPicker.DialogAction" |
+ enum="PhotoPickerDialogAction"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary> |
+ Records which action the user takes in the PhotoPickerDialog. |
Mark P
2017/06/02 17:17:58
Do you have a normalizing action or histogram that
Finnur
2017/06/06 14:23:16
Not sure I need it. The main purpose of this metri
Mark P
2017/06/07 17:19:31
Acknowledged. Good to know that the current list
|
+ </summary> |
+</histogram> |
+ |
+<histogram name="Android.PhotoPicker.ImageByteCount" units="KB"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary>Measures the byte count of a decoded image.</summary> |
Mark P
2017/06/02 17:17:58
ditto my comments on ImageDecodeTime
Finnur
2017/06/06 14:23:16
Done (see answer below).
|
+</histogram> |
+ |
+<histogram name="Android.PhotoPicker.ImageDecodeTime" units="ms"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary> |
+ Measures the amount of time it takes the decoder to decode one image. |
Mark P
2017/06/02 17:17:58
Only recorded on success or always recorded?
Also
Finnur
2017/06/06 14:23:16
RequestProcessTime is for measuring the length of
|
+ </summary> |
+</histogram> |
+ |
+<histogram name="Android.PhotoPicker.RequestProcessTime" units="ms"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary> |
+ Measures the amount of time it takes to process one decode request end to |
+ end (RPC and actual decoding of the bits by utility process). |
Mark P
2017/06/02 17:17:58
Only recorded on success or always recorded?
Finnur
2017/06/06 14:23:16
Done.
|
+ </summary> |
+</histogram> |
+ |
+<histogram name="Android.PhotoPicker.UpscaleLowResBitmap" units="ms"> |
+ <owner>finnur@chromium.org</owner> |
+ <owner>twellington@chromium.org</owner> |
+ <summary> |
+ Measures the amount of time it takes to upscale a bitmap from the low-res |
+ cache. |
Mark P
2017/06/02 17:17:58
optional nit: explain roughly under what condition
Finnur
2017/06/06 14:23:16
Revised.
|
+ </summary> |
+</histogram> |
+ |
<histogram name="Android.PrepareMenu.OpenWebApkVisibilityCheck" units="ms"> |
<owner>hanxi@chromium.org</owner> |
<owner>pkotwicz@chromium.org</owner> |