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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2804283003: [LargeIconService] Report size of the downloaded favicon to UMA (Closed)
Patch Set: Fix the unit-tests #2 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
« no previous file with comments | « components/favicon/core/large_icon_service_unittest.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 23ec2bb552cd4ff0461074355a6abdb026c978b7..178edb8325e4ee89afd156afcd747147f7c28c7a 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -20444,6 +20444,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Favicons.LargeIconService.DownloadedSize" units="pixels">
+ <owner>jkrcal@chromium.org</owner>
+ <summary>
+ Records the size (concretely the width) in pixel of the favicon downloaded
+ from Google favicon server (size 0 denotes that download has failed).
+ </summary>
+</histogram>
+
<histogram name="FileBrowser.CloudImport.UserAction"
enum="CloudImportUserAction">
<owner>smckay@google.com</owner>
« no previous file with comments | « components/favicon/core/large_icon_service_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698