Add UMA for autoblocking and embargoing.

chrome/browser/permissions/permission_uma_util.h

 // 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.
 
 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_
 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_
 
 #include <vector>
 
 #include "base/logging.h"
 #include "base/macros.h"
 #include "chrome/browser/permissions/permission_request.h"
 #include "chrome/browser/permissions/permission_util.h"
 
 enum class PermissionRequestGestureType;
 class GURL;
 class PermissionRequest;
 class Profile;
 
 namespace content {
 enum class PermissionType;
 }  // namespace content
 
+namespace base {
+class TimeDelta;
+}  // namespace base
+
 // This should stay in sync with the SourceUI enum in the permission report
 // protobuf (src/chrome/common/safe_browsing/permission_report.proto).
 enum class PermissionSourceUI {
   PROMPT = 0,
   OIB = 1,
   SITE_SETTINGS = 2,
   PAGE_ACTION = 3,
 
   // Always keep this at the end.
   SOURCE_UI_NUM,
 };
 
 // This should stay in sync with the PersistDecision enum in the permission
 // report message (src/chrome/common/safe_browsing/permission_report.proto).
 enum class PermissionPersistDecision {
   UNSPECIFIED = 0,
   PERSISTED = 1,
   NOT_PERSISTED = 2,
 };
 
+// This is used to back a UMA histogram, so it should be treated as append-only.
+// Any new values should be inserted immediately prior to REPEATED_DISMISSALS.
+enum PermissionEmbargoReason {
+  REPEATED_DISMISSALS = 0,
+  PERMISSIONS_BLACKLISTING = 1,

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

This can probably just be called BLACKLISTED

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

I get a name collision with SafeBrowsingResponse, so it seemed easiest to just
change the name of the embargo reason. Is there a better solution? I know I can
use an enum class but then how do I record that in the metric?

[dominickn, 2017/01/30 00:06:20]

Ah, I see. Leave it as is then. :)

[meredithl, 2017/01/30 00:34:33]

Acknowledged.

+  NOT_EMBARGOED = 2,
+
+  // Always keep this at the end.
+  EMBARGO_NUM,

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

REASON_NUM

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

Done.

+};
+
+enum SafeBrowsingResponse {
+  NOT_BLACKLISTED = 0,
+  TIMEOUT = 1,
+  BLACKLISTED = 2,
+
+  // Always keep this at the end.
+  RESPONSE_NUM,
+};
+
 // A bundle for the information sent in a PermissionReport.
 struct PermissionReportInfo {
   PermissionReportInfo(
       const GURL& origin,
       content::PermissionType permission,
       PermissionAction action,
       PermissionSourceUI source_ui,
       PermissionRequestGestureType gesture_type,
       PermissionPersistDecision persist_decision,
       int num_prior_dismissals,
@@ skipping 53 matching lines @@
                                   const GURL& requesting_origin,
                                   Profile* profile);
   static void PermissionIgnored(content::PermissionType permission,
                                 PermissionRequestGestureType gesture_type,
                                 const GURL& requesting_origin,
                                 Profile* profile);
   static void PermissionRevoked(content::PermissionType permission,
                                 PermissionSourceUI source_ui,
                                 const GURL& revoked_origin,
                                 Profile* profile);
-
+  static void RecordPermissionEmbargoReason(
+      PermissionEmbargoReason embargo_reason);
+  static void RecordSafeBrowsingResponse(base::TimeDelta response_time,
+                                         SafeBrowsingResponse response);
+  static void RecordRepeatedEmbargo(PermissionEmbargoReason embargo_reason);
   // UMA specifically for when permission prompts are shown. This should be
   // roughly equivalent to the metrics above, however it is
   // useful to have separate UMA to a few reasons:
   // - to account for, and get data on coalesced permission bubbles
   // - there are other types of permissions prompts (e.g. download limiting)
   //   which don't go through PermissionContext
   // - the above metrics don't always add up (e.g. sum of
   //   granted+denied+dismissed+ignored is not equal to requested), so it is
   //   unclear from those metrics alone how many prompts are seen by users.
   static void PermissionPromptShown(
@@ skipping 53 matching lines @@
   // for a single origin using |prefix| for the metric.
   static void RecordPermissionPromptPriorCount(
       content::PermissionType permission,
       const std::string& prefix,
       int count);
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionUmaUtil);
 };
 
 #endif  // CHROME_BROWSER_PERMISSIONS_PERMISSION_UMA_UTIL_H_