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

Unified Diff: chrome/browser/content_settings/chrome_content_settings_utils.cc

Issue 2588273002: Add UMA metrics for pop-up blocked page action on desktop (Closed)
Patch Set: Changed the comment of enum PopupsAction Created 3 years, 11 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:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/content_settings/chrome_content_settings_utils.cc
diff --git a/chrome/browser/content_settings/chrome_content_settings_utils.cc b/chrome/browser/content_settings/chrome_content_settings_utils.cc
index c9f5553d4cc511c5fe8c3efdecef4ef55cc02c34..663e7c175d903e6739e44bdff6cfabf39fde4703 100644
--- a/chrome/browser/content_settings/chrome_content_settings_utils.cc
+++ b/chrome/browser/content_settings/chrome_content_settings_utils.cc
@@ -18,4 +18,9 @@ void RecordPluginsAction(PluginsAction action) {
PLUGINS_ACTION_COUNT);
}
+void RecordPopupsAction(PopupsAction action) {
+ UMA_HISTOGRAM_ENUMERATION("ContentSettings.Popups", action,
+ POPUPS_ACTION_COUNT);
+}
+
} // namespace content_settings

Powered by Google App Engine
This is Rietveld 408576698