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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 1819503002: Separate SBClientDownload.SignedOrWhitelistedDownload metric into two buckets, (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add another bucket Created 4 years, 9 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 ba8aa2e7d974703b9e82056b9b0ae5905c399184..597d0b9c7b9541b6916b02b871161cdfee226dbd 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -43538,6 +43538,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</histogram>
<histogram name="SBClientDownload.SignedOrWhitelistedDownload">
+ <obsolete>
+ Deprecated in Chrome 50. Replaced by SBClientDownload.WhitelistedDownload.*.
+ </obsolete>
<owner>mattm@chromium.org</owner>
<summary>
Counter which is incremented whenever an executable is downloaded which is
@@ -43545,6 +43548,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="SBClientDownload.WhitelistedDownload"
+ enum="WhitelistedDownloadType">
+ <owner>jialiul@chromium.org</owner>
+ <summary>
+ When a download starts, record if it matches a certain whitelist (e.g. URL
+ whitelist, or signature whitelist).
mattm 2016/03/21 20:57:19 Should more precisely define what this histogram m
Jialiu Lin 2016/03/21 21:26:48 You're right. Updated accordingly. Thanks!
+ </summary>
+</histogram>
+
<histogram name="SBClientDownload.ZipFileHasArchiveButNoExecutable"
enum="Boolean">
<owner>mattm@chromium.org</owner>
@@ -84173,6 +84185,12 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="1" label="Revoked ephemeral permission"/>
</enum>
+<enum name="WhitelistedDownloadType" type="int">
+ <int value="0" label="Does not match any whitelists"/>
+ <int value="1" label="URL whitelist"/>
+ <int value="2" label="Signature whitelist"/>
+</enum>
+
<enum name="WiFiApMode" type="int">
<int value="0" label="Unknown"/>
<int value="1" label="Managed"/>

Powered by Google App Engine
This is Rietveld 408576698