Add UMA for recording embargo reasons and autoblocker interactions.

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"
@@ skipping 47 matching lines @@
   GURL origin;
   content::PermissionType permission;
   PermissionAction action;
   PermissionSourceUI source_ui;
   PermissionRequestGestureType gesture_type;
   PermissionPersistDecision persist_decision;
   int num_prior_dismissals;
   int num_prior_ignores;
 };
 
+enum PermissionEmbargoReason {

[kcarattini, 2017/02/09 00:17:33]

Nit: What about PermissionEmbargoStatus instead of "reason"? I think reason
sounds a bit odd in the case where it is not embargoed.

[meredithl, 2017/02/09 02:58:28]

Done.

+  NOT_EMBARGOED = 0,
+  BLACKLISTED = 1,
+  REPEATED_DISMISSALS = 2,
+
+  // Keep this at the end.
+  REASON_NUM,
+};
+
 // Provides a convenient way of logging UMA for permission related operations.
 class PermissionUmaUtil {
  public:
   static const char kPermissionsPromptShown[];
   static const char kPermissionsPromptShownGesture[];
   static const char kPermissionsPromptShownNoGesture[];
   static const char kPermissionsPromptAccepted[];
   static const char kPermissionsPromptAcceptedGesture[];
   static const char kPermissionsPromptAcceptedNoGesture[];
   static const char kPermissionsPromptDenied[];
@@ skipping 29 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);

[raymes, 2017/02/08 23:20:09]

nit: please add a newline after this

[meredithl, 2017/02/09 00:15:37]

Done.

   // 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_