Add UMA for autoblocking and embargoing.

chrome/browser/permissions/permission_uma_util.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/permissions/permission_uma_util.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/stringprintf.h"
+#include "base/time/time.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/permissions/permission_decision_auto_blocker.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/common/chrome_switches.h"
@@ skipping 29 matching lines @@
                                            no_gesture_metric_name,           \
                                            gesture_type,                     \
                                            permission_bubble_type)           \
   if (gesture_type == PermissionRequestGestureType::GESTURE) {               \
     PERMISSION_BUBBLE_TYPE_UMA(gesture_metric_name, permission_bubble_type); \
   } else if (gesture_type == PermissionRequestGestureType::NO_GESTURE) {     \
     PERMISSION_BUBBLE_TYPE_UMA(no_gesture_metric_name,                       \
                                permission_bubble_type);                      \
   }
 
+#define PERMISSION_UMA

[dominickn, 2017/01/27 06:05:47]

Remove this :)

[meredithl, 2017/01/29 23:48:30]

Done.

+
 using content::PermissionType;
 
 namespace {
 
 const std::string GetRapporMetric(PermissionType permission,
                                   PermissionAction action) {
   std::string action_str;
   switch (action) {
     case GRANTED:
       action_str = "Granted";
@@ skipping 279 matching lines @@
       permission == PermissionType::AUDIO_CAPTURE ||
       permission == PermissionType::VIDEO_CAPTURE) {
     // An unknown gesture type is passed in since gesture type is only
     // applicable in prompt UIs where revocations are not possible.
     RecordPermissionAction(permission, REVOKED, source_ui,
                            PermissionRequestGestureType::UNKNOWN,
                            revoked_origin, profile);
   }
 }
 
+void PermissionUmaUtil::RecordPermissionEmbargoReason(
+    PermissionEmbargoReason embargo_reason) {
+  UMA_HISTOGRAM_ENUMERATION("Permissions.AutoBlocker.EmbargoReason",
+                            embargo_reason,
+                            PermissionEmbargoReason::EMBARGO_NUM);
+}
+
+void PermissionUmaUtil::RecordRepeatedEmbargo(
+    PermissionEmbargoReason embargo_reason) {
+  // NOT_EMBARGOED should not ever result in an (origin, permission) pair being
+  // placed under embargo, let alone repeatedly.
+  DCHECK_NE(PermissionEmbargoReason::NOT_EMBARGOED, embargo_reason);
+  UMA_HISTOGRAM_ENUMERATION("Permissions.AutoBlocker.RepeatedEmbargo",
+                            embargo_reason,
+                            PermissionEmbargoReason::EMBARGO_NUM);
+}
+
+void PermissionUmaUtil::RecordSafeBrowsingResponse(
+    base::TimeDelta response_time,
+    SafeBrowsingResponse response) {
+  UMA_HISTOGRAM_TIMES("Permissions.AutoBlocker.SafeBrowsingResponseTime",
+                      response_time);
+  UMA_HISTOGRAM_ENUMERATION("Permissions.AutoBlocker.SafeBrowsingResponse",
+                            response, SafeBrowsingResponse::RESPONSE_NUM);
+}
+
 void PermissionUmaUtil::PermissionPromptShown(
     const std::vector<PermissionRequest*>& requests) {
   DCHECK(!requests.empty());
 
   PermissionRequestType permission_prompt_type =
       PermissionRequestType::MULTIPLE;
   PermissionRequestGestureType permission_gesture_type =
       PermissionRequestGestureType::UNKNOWN;
   if (requests.size() == 1) {
     permission_prompt_type = requests[0]->GetPermissionRequestType();
@@ skipping 346 matching lines @@
   if (!deprecated_metric.empty() && rappor_service) {
     rappor::SampleDomainAndRegistryFromGURL(rappor_service, deprecated_metric,
                                             requesting_origin);
 
     std::string rappor_metric = deprecated_metric + "2";
     rappor_service->RecordSampleString(
         rappor_metric, rappor::LOW_FREQUENCY_ETLD_PLUS_ONE_RAPPOR_TYPE,
         rappor::GetDomainAndRegistrySampleFromGURL(requesting_origin));
   }
 }