| OLD | NEW |
| 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_context_uma_util.h" | 5 #include "chrome/browser/permissions/permission_context_uma_util.h" |
| 6 | 6 |
| 7 #include "base/metrics/histogram_macros.h" | 7 #include "base/metrics/histogram_macros.h" |
| 8 #include "base/strings/stringprintf.h" | 8 #include "base/strings/stringprintf.h" |
| 9 #include "chrome/browser/browser_process.h" | 9 #include "chrome/browser/browser_process.h" |
| 10 #include "chrome/browser/permissions/permission_manager.h" | 10 #include "chrome/browser/permissions/permission_manager.h" |
| (...skipping 262 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 273 | 273 |
| 274 // In order to gauge the compatibility risk of implementing an improved | 274 // In order to gauge the compatibility risk of implementing an improved |
| 275 // iframe permissions security model, we would like to know the ratio of | 275 // iframe permissions security model, we would like to know the ratio of |
| 276 // same-origin to cross-origin permission requests. Our estimate of this | 276 // same-origin to cross-origin permission requests. Our estimate of this |
| 277 // ratio could be somewhat biased by repeated requests coming from a | 277 // ratio could be somewhat biased by repeated requests coming from a |
| 278 // single frame, but we expect this to be insignificant. | 278 // single frame, but we expect this to be insignificant. |
| 279 if (requesting_origin.GetOrigin() != embedding_origin.GetOrigin()) { | 279 if (requesting_origin.GetOrigin() != embedding_origin.GetOrigin()) { |
| 280 content::PermissionManager* manager = profile->GetPermissionManager(); | 280 content::PermissionManager* manager = profile->GetPermissionManager(); |
| 281 if (!manager) | 281 if (!manager) |
| 282 return; | 282 return; |
| 283 content::PermissionStatus embedding_permission_status = | 283 permission::Status embedding_permission_status = |
| 284 manager->GetPermissionStatus(type, embedding_origin, embedding_origin); | 284 manager->GetPermissionStatus(type, embedding_origin, embedding_origin); |
| 285 | 285 |
| 286 base::HistogramBase* histogram = base::LinearHistogram::FactoryGet( | 286 base::HistogramBase* histogram = base::LinearHistogram::FactoryGet( |
| 287 "Permissions.Requested.CrossOrigin_" + PermissionTypeToString(type), 1, | 287 "Permissions.Requested.CrossOrigin_" + PermissionTypeToString(type), 1, |
| 288 content::PERMISSION_STATUS_LAST, content::PERMISSION_STATUS_LAST + 1, | 288 permission::STATUS_LAST, permission::STATUS_LAST + 1, |
| 289 base::HistogramBase::kUmaTargetedHistogramFlag); | 289 base::HistogramBase::kUmaTargetedHistogramFlag); |
| 290 histogram->Add(embedding_permission_status); | 290 histogram->Add(embedding_permission_status); |
| 291 } else { | 291 } else { |
| 292 UMA_HISTOGRAM_ENUMERATION( | 292 UMA_HISTOGRAM_ENUMERATION( |
| 293 "Permissions.Requested.SameOrigin", | 293 "Permissions.Requested.SameOrigin", |
| 294 static_cast<base::HistogramBase::Sample>(type), | 294 static_cast<base::HistogramBase::Sample>(type), |
| 295 static_cast<base::HistogramBase::Sample>(PermissionType::NUM)); | 295 static_cast<base::HistogramBase::Sample>(PermissionType::NUM)); |
| 296 } | 296 } |
| 297 } | 297 } |
| 298 | 298 |
| (...skipping 22 matching lines...) Expand all Loading... |
| 321 | 321 |
| 322 void PermissionContextUmaUtil::PermissionDismissed( | 322 void PermissionContextUmaUtil::PermissionDismissed( |
| 323 ContentSettingsType permission, const GURL& requesting_origin) { | 323 ContentSettingsType permission, const GURL& requesting_origin) { |
| 324 RecordPermissionAction(permission, DISMISSED, requesting_origin); | 324 RecordPermissionAction(permission, DISMISSED, requesting_origin); |
| 325 } | 325 } |
| 326 | 326 |
| 327 void PermissionContextUmaUtil::PermissionIgnored( | 327 void PermissionContextUmaUtil::PermissionIgnored( |
| 328 ContentSettingsType permission, const GURL& requesting_origin) { | 328 ContentSettingsType permission, const GURL& requesting_origin) { |
| 329 RecordPermissionAction(permission, IGNORED, requesting_origin); | 329 RecordPermissionAction(permission, IGNORED, requesting_origin); |
| 330 } | 330 } |
| OLD | NEW |