Add UMA for recording embargo reasons and autoblocker interactions.

chrome/browser/permissions/permission_queue_controller.cc

Index: chrome/browser/permissions/permission_queue_controller.cc
diff --git a/chrome/browser/permissions/permission_queue_controller.cc b/chrome/browser/permissions/permission_queue_controller.cc
index 4a126626edb7195011b91c3f910c805f7e4c6884..90b17a3a9cc64c88cb87f94da47d6c1b4e380417 100644
--- a/chrome/browser/permissions/permission_queue_controller.cc
+++ b/chrome/browser/permissions/permission_queue_controller.cc
@@ -228,16 +228,28 @@ void PermissionQueueController::OnPermissionSet(const PermissionRequestID& id,
                                            requesting_frame, profile_);
       PermissionUmaUtil::RecordPermissionPromptAccepted(request_type,
                                                         gesture_type);
+      PermissionUmaUtil::RecordPermissionEmbargoStatus(
+          PermissionEmbargoStatus::NOT_EMBARGOED);
       break;
     case DENIED:
       PermissionUmaUtil::PermissionDenied(permission_type_, gesture_type,
                                           requesting_frame, profile_);
       PermissionUmaUtil::RecordPermissionPromptDenied(request_type,
                                                       gesture_type);
+      PermissionUmaUtil::RecordPermissionEmbargoStatus(
+          PermissionEmbargoStatus::NOT_EMBARGOED);
       break;
     case DISMISSED:
       PermissionUmaUtil::PermissionDismissed(permission_type_, gesture_type,
                                              requesting_frame, profile_);
+      if (PermissionDecisionAutoBlocker::GetForProfile(profile_)
+              ->RecordDismissAndEmbargo(requesting_origin, permission_type_)) {

[raymes, 2017/02/09 05:46:39]

Unfortunately this is going to be untested code :( I think we would want to make
sure we have tests before shipping this to stable, which we should get for free
after the refactoring is done. 

I'm less concerned about the metrics, though it's good to have tests for those
too. 

dominickn@ what do you think?

[dominickn, 2017/02/10 03:12:33]

We could look at adding Java side tests for this. That should be fairly
straightforward.

+        PermissionUmaUtil::RecordPermissionEmbargoStatus(
+            PermissionEmbargoStatus::REPEATED_DISMISSALS);
+      } else {
+        PermissionUmaUtil::RecordPermissionEmbargoStatus(
+            PermissionEmbargoStatus::NOT_EMBARGOED);
+      }
       break;
     default:
       NOTREACHED();