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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/permissions/permission_uma_util.h" 5 #include "chrome/browser/permissions/permission_uma_util.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/metrics/histogram_macros.h" 9 #include "base/metrics/histogram_macros.h"
10 #include "base/strings/stringprintf.h" 10 #include "base/strings/stringprintf.h"
11 #include "chrome/browser/browser_process.h" 11 #include "chrome/browser/browser_process.h"
12 #include "chrome/browser/permissions/permission_manager.h" 12 #include "chrome/browser/permissions/permission_manager.h"
13 #include "chrome/browser/permissions/permission_util.h" 13 #include "chrome/browser/permissions/permission_util.h"
14 #include "chrome/browser/profiles/profile.h" 14 #include "chrome/browser/profiles/profile.h"
15 #include "chrome/browser/safe_browsing/permission_reporter.h"
16 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
17 #include "chrome/browser/safe_browsing/ui_manager.h"
15 #include "chrome/browser/ui/website_settings/permission_bubble_request.h" 18 #include "chrome/browser/ui/website_settings/permission_bubble_request.h"
16 #include "components/rappor/rappor_service.h" 19 #include "components/rappor/rappor_service.h"
17 #include "components/rappor/rappor_utils.h" 20 #include "components/rappor/rappor_utils.h"
18 #include "content/public/browser/permission_type.h" 21 #include "content/public/browser/permission_type.h"
19 #include "content/public/common/origin_util.h" 22 #include "content/public/common/origin_util.h"
20 #include "url/gurl.h" 23 #include "url/gurl.h"
21 24
22 // UMA keys need to be statically initialized so plain function would not 25 // UMA keys need to be statically initialized so plain function would not
23 // work. Use macros instead. 26 // work. Use macros instead.
24 #define PERMISSION_ACTION_UMA(secure_origin, permission, permission_secure, \ 27 #define PERMISSION_ACTION_UMA(secure_origin, permission, permission_secure, \
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 PermissionUtil::GetPermissionString(permission); 73 PermissionUtil::GetPermissionString(permission);
71 if (permission_str.empty()) 74 if (permission_str.empty())
72 return ""; 75 return "";
73 return base::StringPrintf("ContentSettings.PermissionActions_%s.%s.Url", 76 return base::StringPrintf("ContentSettings.PermissionActions_%s.%s.Url",
74 permission_str.c_str(), action_str.c_str()); 77 permission_str.c_str(), action_str.c_str());
75 } 78 }
76 79
77 void RecordPermissionAction(PermissionType permission, 80 void RecordPermissionAction(PermissionType permission,
78 PermissionAction action, 81 PermissionAction action,
79 const GURL& requesting_origin) { 82 const GURL& requesting_origin) {
83 // Send permission action report to opted-in users.
84 g_browser_process->safe_browsing_service()
85 ->ui_manager()
86 ->ReportPermissionAction(requesting_origin, permission, action);
87
80 bool secure_origin = content::IsOriginSecure(requesting_origin); 88 bool secure_origin = content::IsOriginSecure(requesting_origin);
81 89
82 switch (permission) { 90 switch (permission) {
83 case PermissionType::GEOLOCATION: 91 case PermissionType::GEOLOCATION:
84 PERMISSION_ACTION_UMA( 92 PERMISSION_ACTION_UMA(
85 secure_origin, 93 secure_origin,
86 "Permissions.Action.Geolocation", 94 "Permissions.Action.Geolocation",
87 "Permissions.Action.SecureOrigin.Geolocation", 95 "Permissions.Action.SecureOrigin.Geolocation",
88 "Permissions.Action.InsecureOrigin.Geolocation", 96 "Permissions.Action.InsecureOrigin.Geolocation",
89 action); 97 action);
(...skipping 277 matching lines...) Expand 10 before | Expand all | Expand 10 after
367 } 375 }
368 376
369 void PermissionUmaUtil::PermissionPromptDenied( 377 void PermissionUmaUtil::PermissionPromptDenied(
370 const std::vector<PermissionBubbleRequest*>& requests) { 378 const std::vector<PermissionBubbleRequest*>& requests) {
371 DCHECK(!requests.empty()); 379 DCHECK(!requests.empty());
372 DCHECK(requests.size() == 1); 380 DCHECK(requests.size() == 1);
373 381
374 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied, 382 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied,
375 requests[0]->GetPermissionBubbleType()); 383 requests[0]->GetPermissionBubbleType());
376 } 384 }
OLDNEW
« 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