Add UMA for autoblocking and embargoing.

chrome/browser/permissions/permission_uma_util.cc

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 82792775140b8897d198d9cf5d976795f766daf7..0d20d6a337f1c3f88d81aa0df9d5172d24faf353 100644
--- a/chrome/browser/permissions/permission_uma_util.cc
+++ b/chrome/browser/permissions/permission_uma_util.cc
@@ -9,6 +9,7 @@
 #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"
@@ -58,6 +59,8 @@
                                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 {
@@ -357,6 +360,32 @@ void PermissionUmaUtil::PermissionRevoked(PermissionType permission,
   }
 }
 
+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());