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

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

Issue 2047253002: Add hooks to permission layer for permission action reporting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@permission-reporter-implementation
Patch Set: Small comment change Created 4 years, 6 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/permissions/permission_uma_util.cc
diff --git a/chrome/browser/permissions/permission_uma_util.cc b/chrome/browser/permissions/permission_uma_util.cc
index efbdf5f04b6c65ff92efbb88a5fa6d95f2c44fb1..900de90c83017ac7d2832f916b3c208daf2dd5e3 100644
--- a/chrome/browser/permissions/permission_uma_util.cc
+++ b/chrome/browser/permissions/permission_uma_util.cc
@@ -12,6 +12,9 @@
#include "chrome/browser/permissions/permission_manager.h"
#include "chrome/browser/permissions/permission_util.h"
#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/safe_browsing/permission_reporter.h"
+#include "chrome/browser/safe_browsing/safe_browsing_service.h"
+#include "chrome/browser/safe_browsing/ui_manager.h"
#include "chrome/browser/ui/website_settings/permission_bubble_request.h"
#include "components/rappor/rappor_service.h"
#include "components/rappor/rappor_utils.h"
@@ -77,6 +80,11 @@ const std::string GetRapporMetric(PermissionType permission,
void RecordPermissionAction(PermissionType permission,
PermissionAction action,
const GURL& requesting_origin) {
+ // Send permission action report to opted-in users.
+ g_browser_process->safe_browsing_service()
+ ->ui_manager()
+ ->ReportPermissionAction(requesting_origin, permission, action);
+
bool secure_origin = content::IsOriginSecure(requesting_origin);
switch (permission) {
« no previous file with comments | « no previous file | chrome/browser/safe_browsing/ping_manager.h » ('j') | chrome/browser/safe_browsing/ping_manager.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698