Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(137)

Unified Diff: tools/metrics/histograms/histograms.xml

Side-by-side diff isn't available for this file because of its large size.
Issue 2806653002: Add UMA to track why a download is non-parallel when enabled parallel (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 790748cfab6013c941832b528f865b61ac5721bc..80f090fdaea2e71b0be5a31500e6689e82db3b0a 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -13865,6 +13865,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Download.ParallelDownloadFailReason"
qinmin 2017/04/07 21:00:52 Download.ParallelDownload.CreationFailureReason?
xingliu 2017/04/08 01:13:28 Done.
+ enum="ParallelDownloadFailReason">
+ <owner>xingliu@chromium.org</owner>
+ <summary>
+ Records why the download is not created as parallel download.
Ilya Sherman 2017/04/07 22:13:08 Please document when this metric is recorded. Is
Ilya Sherman 2017/04/07 22:13:09 I'd recommend including "success" as bucket zero i
+ </summary>
+</histogram>
+
<histogram name="Download.ParallelDownloadRequestCount" units="requests">
<owner>xingliu@chromium.org</owner>
<summary>
@@ -105714,6 +105722,19 @@ from previous Chrome versions.
<int value="1" label="HRTF"/>
</enum>
+<enum name="ParallelDownloadFailReason" type="int">
+ <summary>
+ The reason why parallel download is not applicable and fallbacks to a
+ non-parallel download.
+ </summary>
+ <int value="0" label="STRONG_VALIDATORS"/>
+ <int value="1" label="ACCEPT_RANGE_HEADER"/>
+ <int value="2" label="CONTENT_LENGTH_HEADER"/>
+ <int value="3" label="FILE_SIZE"/>
+ <int value="4" label="CONNECTION_TYPE"/>
+ <int value="5" label="FAIL_COUNT"/>
+</enum>
+
<enum name="ParsedCookieStatus" type="int">
<obsolete>
Deprecated as of 9/2013. Experiment to measure control characters in cookies

Powered by Google App Engine
This is Rietveld 408576698