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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 2160003002: Add UMA to record why chrome shows generic warning (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add UMA for dangerour file reason Created 4 years, 5 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
« no previous file with comments | « chrome/browser/download/chrome_download_manager_delegate.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index e90336494d96df158b3cc62e6eb612df59bc7b23..646127c8b731f400a7236c2506460a20fc0fc51c 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -10602,6 +10602,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Download.DangerousFile.Reason" enum="DangerousFile.Reason">
+ <owner>jialiul@chromium.org</owner>
+ <owner>nparker@chromium.org</owner>
+ <summary>
+ Indicates why a download is mared as DANGEROUS_FILE. Grouped by reason, such
Nathan Parker 2016/07/18 21:21:38 s/mared/marked Can you add a comment about how th
Jialiu Lin 2016/07/18 21:45:18 Done
+ as Safe Browsing (SB) service is not available, and SB returns UNKOWN or
+ SAFE verdict.
+ </summary>
+</histogram>
+
<histogram name="Download.DangerousFile.UserDiscard"
enum="DownloadItem.DangerousFileType">
<owner>asanka@chromium.org</owner>
@@ -70410,6 +70420,12 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<int value="2" label="Clock Changed"/>
</enum>
+<enum name="DangerousFile.Reason" type="int">
+ <int value="0" label="Safe Browsing is not available"/>
+ <int value="1" label="Safe Browsing returns UNKOWN"/>
+ <int value="2" label="Safe Browsing returns SAFE"/>
+</enum>
+
<enum name="DarkResumeScanRetryResult" type="int">
<int value="0" label="Not Connected"/>
<int value="1" label="Connected"/>
« no previous file with comments | « chrome/browser/download/chrome_download_manager_delegate.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698