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

Unified Diff: chrome/browser/ui/website_settings/permission_ui_uma_util.cc

Issue 1884073002: Add a new metric for how many times permissions prompts are shown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Feedback Created 4 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:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/website_settings/permission_ui_uma_util.cc
diff --git a/chrome/browser/ui/website_settings/permission_ui_uma_util.cc b/chrome/browser/ui/website_settings/permission_ui_uma_util.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b97b05f9079d3c847b60745da0c48bf7a45891b5
--- /dev/null
+++ b/chrome/browser/ui/website_settings/permission_ui_uma_util.cc
@@ -0,0 +1,19 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/website_settings/permission_ui_uma_util.h"
+
+#include "base/metrics/histogram_macros.h"
+#include "chrome/browser/ui/website_settings/permission_bubble_request.h"
+
+void PermissionUiUmaUtil::PermissionPromptShown(
+ const std::vector<PermissionBubbleRequest*>& requests) {
+ PermissionBubbleType permission_prompt_type = PermissionBubbleType::MULTIPLE;
+ if (requests.size() == 1)
+ permission_prompt_type = requests[0]->GetPermissionBubbleType();
+ UMA_HISTOGRAM_ENUMERATION(
+ "Permissions.Prompt.Shown",
+ static_cast<base::HistogramBase::Sample>(permission_prompt_type),
+ static_cast<base::HistogramBase::Sample>(PermissionBubbleType::NUM));
+}
« no previous file with comments | « chrome/browser/ui/website_settings/permission_bubble_request.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698