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

Side by Side 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 Android 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 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_request.h"
13 #include "chrome/browser/permissions/permission_util.h" 14 #include "chrome/browser/permissions/permission_util.h"
14 #include "chrome/browser/profiles/profile.h" 15 #include "chrome/browser/profiles/profile.h"
15 #include "chrome/browser/ui/website_settings/permission_bubble_request.h" 16 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
17 #include "chrome/browser/safe_browsing/ui_manager.h"
18 #include "chrome/browser/sync/profile_sync_service_factory.h"
19 #include "chrome/common/chrome_switches.h"
20 #include "chrome/common/pref_names.h"
21 #include "components/browser_sync/browser/profile_sync_service.h"
22 #include "components/prefs/pref_service.h"
16 #include "components/rappor/rappor_service.h" 23 #include "components/rappor/rappor_service.h"
17 #include "components/rappor/rappor_utils.h" 24 #include "components/rappor/rappor_utils.h"
18 #include "content/public/browser/permission_type.h" 25 #include "content/public/browser/permission_type.h"
19 #include "content/public/common/origin_util.h" 26 #include "content/public/common/origin_util.h"
20 #include "url/gurl.h" 27 #include "url/gurl.h"
21 28
22 // UMA keys need to be statically initialized so plain function would not 29 // UMA keys need to be statically initialized so plain function would not
23 // work. Use macros instead. 30 // work. Use macros instead.
24 #define PERMISSION_ACTION_UMA(secure_origin, permission, permission_secure, \ 31 #define PERMISSION_ACTION_UMA(secure_origin, permission, permission_secure, \
25 permission_insecure, action) \ 32 permission_insecure, action) \
(...skipping 279 matching lines...) Expand 10 before | Expand all | Expand 10 after
305 // permissions. 312 // permissions.
306 if (permission == PermissionType::NOTIFICATIONS || 313 if (permission == PermissionType::NOTIFICATIONS ||
307 permission == PermissionType::GEOLOCATION || 314 permission == PermissionType::GEOLOCATION ||
308 permission == PermissionType::AUDIO_CAPTURE || 315 permission == PermissionType::AUDIO_CAPTURE ||
309 permission == PermissionType::VIDEO_CAPTURE) { 316 permission == PermissionType::VIDEO_CAPTURE) {
310 RecordPermissionAction(permission, REVOKED, revoked_origin); 317 RecordPermissionAction(permission, REVOKED, revoked_origin);
311 } 318 }
312 } 319 }
313 320
314 void PermissionUmaUtil::PermissionPromptShown( 321 void PermissionUmaUtil::PermissionPromptShown(
315 const std::vector<PermissionBubbleRequest*>& requests) { 322 const std::vector<PermissionRequest*>& requests) {
316 DCHECK(!requests.empty()); 323 DCHECK(!requests.empty());
317 324
318 PermissionBubbleType permission_prompt_type = PermissionBubbleType::MULTIPLE; 325 PermissionBubbleType permission_prompt_type = PermissionBubbleType::MULTIPLE;
319 if (requests.size() == 1) 326 if (requests.size() == 1)
320 permission_prompt_type = requests[0]->GetPermissionBubbleType(); 327 permission_prompt_type = requests[0]->GetPermissionBubbleType();
321 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptShown, permission_prompt_type); 328 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptShown, permission_prompt_type);
322 329
323 UMA_HISTOGRAM_ENUMERATION( 330 UMA_HISTOGRAM_ENUMERATION(
324 kPermissionsPromptRequestsPerPrompt, 331 kPermissionsPromptRequestsPerPrompt,
325 static_cast<base::HistogramBase::Sample>(requests.size()), 332 static_cast<base::HistogramBase::Sample>(requests.size()),
326 static_cast<base::HistogramBase::Sample>(10)); 333 static_cast<base::HistogramBase::Sample>(10));
327 334
328 if (requests.size() > 1) { 335 if (requests.size() > 1) {
329 for (const auto* request : requests) { 336 for (const auto* request : requests) {
330 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptMergedBubbleTypes, 337 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptMergedBubbleTypes,
331 request->GetPermissionBubbleType()); 338 request->GetPermissionBubbleType());
332 } 339 }
333 } 340 }
334 } 341 }
335 342
336 void PermissionUmaUtil::PermissionPromptAccepted( 343 void PermissionUmaUtil::PermissionPromptAccepted(
337 const std::vector<PermissionBubbleRequest*>& requests, 344 const std::vector<PermissionRequest*>& requests,
338 const std::vector<bool>& accept_states) { 345 const std::vector<bool>& accept_states) {
339 DCHECK(!requests.empty()); 346 DCHECK(!requests.empty());
340 DCHECK(requests.size() == accept_states.size()); 347 DCHECK(requests.size() == accept_states.size());
341 348
342 bool all_accepted = accept_states[0]; 349 bool all_accepted = accept_states[0];
343 PermissionBubbleType permission_prompt_type = 350 PermissionBubbleType permission_prompt_type =
344 requests[0]->GetPermissionBubbleType(); 351 requests[0]->GetPermissionBubbleType();
345 if (requests.size() > 1) { 352 if (requests.size() > 1) {
346 permission_prompt_type = PermissionBubbleType::MULTIPLE; 353 permission_prompt_type = PermissionBubbleType::MULTIPLE;
347 for (size_t i = 0; i < requests.size(); ++i) { 354 for (size_t i = 0; i < requests.size(); ++i) {
(...skipping 12 matching lines...) Expand all
360 if (all_accepted) { 367 if (all_accepted) {
361 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptAccepted, 368 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptAccepted,
362 permission_prompt_type); 369 permission_prompt_type);
363 } else { 370 } else {
364 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied, 371 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied,
365 permission_prompt_type); 372 permission_prompt_type);
366 } 373 }
367 } 374 }
368 375
369 void PermissionUmaUtil::PermissionPromptDenied( 376 void PermissionUmaUtil::PermissionPromptDenied(
370 const std::vector<PermissionBubbleRequest*>& requests) { 377 const std::vector<PermissionRequest*>& requests) {
371 DCHECK(!requests.empty()); 378 DCHECK(!requests.empty());
372 DCHECK(requests.size() == 1); 379 DCHECK(requests.size() == 1);
373 380
374 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied, 381 PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied,
375 requests[0]->GetPermissionBubbleType()); 382 requests[0]->GetPermissionBubbleType());
376 } 383 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698