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

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: Created 4 years 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..5c3c9e9ecd6cb969b402025bb72d80a23076e8a1 100644
--- a/chrome/browser/content_settings/chrome_content_settings_utils.cc
+++ b/chrome/browser/content_settings/chrome_content_settings_utils.cc
@@ -18,4 +18,10 @@ void RecordPluginsAction(PluginsAction action) {
PLUGINS_ACTION_COUNT);
}
+void RecordPopupsAction(PopupsAction action) {
+ UMA_HISTOGRAM_ENUMERATION("ContentSettings.Popups", action,
+ POPUPS_ACTION_COUNT);
+}
+
dominickn 2016/12/20 05:31:48 Nit: remove the extra newline here (have only one)
charleszhao 2016/12/20 22:43:50 Done.
+
} // namespace content_settings

Powered by Google App Engine
This is Rietveld 408576698