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

Unified Diff: chrome/browser/permissions/permission_uma_util.cc

Issue 2123653006: Rename PermissionBubbleRequest to PermissionRequest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@permission_manager_rename
Patch Set: Fix missed file 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:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/permissions/permission_uma_util.h ('k') | chrome/browser/ui/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/permissions/permission_uma_util.cc
diff --git a/chrome/browser/permissions/permission_uma_util.cc b/chrome/browser/permissions/permission_uma_util.cc
index 70324dd516908e38589b4eabab7780f1869396de..e73a17632e3bd15c207f511875ae0567143d52c3 100644
--- a/chrome/browser/permissions/permission_uma_util.cc
+++ b/chrome/browser/permissions/permission_uma_util.cc
@@ -11,12 +11,12 @@
#include "base/strings/stringprintf.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/permissions/permission_manager.h"
+#include "chrome/browser/permissions/permission_request.h"
#include "chrome/browser/permissions/permission_util.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/safe_browsing/safe_browsing_service.h"
#include "chrome/browser/safe_browsing/ui_manager.h"
#include "chrome/browser/sync/profile_sync_service_factory.h"
-#include "chrome/browser/ui/website_settings/permission_bubble_request.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/pref_names.h"
#include "components/browser_sync/browser/profile_sync_service.h"
@@ -44,7 +44,7 @@
UMA_HISTOGRAM_ENUMERATION( \
metric_name, \
static_cast<base::HistogramBase::Sample>(permission_bubble_type), \
- static_cast<base::HistogramBase::Sample>(PermissionBubbleType::NUM))
+ static_cast<base::HistogramBase::Sample>(PermissionRequestType::NUM))
using content::PermissionType;
@@ -232,12 +232,13 @@ void PermissionUmaUtil::PermissionRevoked(PermissionType permission,
}
void PermissionUmaUtil::PermissionPromptShown(
- const std::vector<PermissionBubbleRequest*>& requests) {
+ const std::vector<PermissionRequest*>& requests) {
DCHECK(!requests.empty());
- PermissionBubbleType permission_prompt_type = PermissionBubbleType::MULTIPLE;
+ PermissionRequestType permission_prompt_type =
+ PermissionRequestType::MULTIPLE;
if (requests.size() == 1)
- permission_prompt_type = requests[0]->GetPermissionBubbleType();
+ permission_prompt_type = requests[0]->GetPermissionRequestType();
PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptShown, permission_prompt_type);
UMA_HISTOGRAM_ENUMERATION(
@@ -248,31 +249,31 @@ void PermissionUmaUtil::PermissionPromptShown(
if (requests.size() > 1) {
for (const auto* request : requests) {
PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptMergedBubbleTypes,
- request->GetPermissionBubbleType());
+ request->GetPermissionRequestType());
}
}
}
void PermissionUmaUtil::PermissionPromptAccepted(
- const std::vector<PermissionBubbleRequest*>& requests,
+ const std::vector<PermissionRequest*>& requests,
const std::vector<bool>& accept_states) {
DCHECK(!requests.empty());
DCHECK(requests.size() == accept_states.size());
bool all_accepted = accept_states[0];
- PermissionBubbleType permission_prompt_type =
- requests[0]->GetPermissionBubbleType();
+ PermissionRequestType permission_prompt_type =
+ requests[0]->GetPermissionRequestType();
if (requests.size() > 1) {
- permission_prompt_type = PermissionBubbleType::MULTIPLE;
+ permission_prompt_type = PermissionRequestType::MULTIPLE;
for (size_t i = 0; i < requests.size(); ++i) {
const auto* request = requests[i];
if (accept_states[i]) {
PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptMergedBubbleAccepted,
- request->GetPermissionBubbleType());
+ request->GetPermissionRequestType());
} else {
all_accepted = false;
PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptMergedBubbleDenied,
- request->GetPermissionBubbleType());
+ request->GetPermissionRequestType());
}
}
}
@@ -287,12 +288,12 @@ void PermissionUmaUtil::PermissionPromptAccepted(
}
void PermissionUmaUtil::PermissionPromptDenied(
- const std::vector<PermissionBubbleRequest*>& requests) {
+ const std::vector<PermissionRequest*>& requests) {
DCHECK(!requests.empty());
DCHECK(requests.size() == 1);
PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied,
- requests[0]->GetPermissionBubbleType());
+ requests[0]->GetPermissionRequestType());
}
bool PermissionUmaUtil::IsOptedIntoPermissionActionReporting(Profile* profile) {
« no previous file with comments | « chrome/browser/permissions/permission_uma_util.h ('k') | chrome/browser/ui/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698