Add UMA for autoblocking and embargoing.

chrome/browser/permissions/permission_blacklist_client.cc

Index: chrome/browser/permissions/permission_blacklist_client.cc
diff --git a/chrome/browser/permissions/permission_blacklist_client.cc b/chrome/browser/permissions/permission_blacklist_client.cc
index ef0be9b38574c4b4515a9923c5d5666b2d5ff6c4..29c052acf5b9f94fe13f4ee4d92921b0dfbc069e 100644
--- a/chrome/browser/permissions/permission_blacklist_client.cc
+++ b/chrome/browser/permissions/permission_blacklist_client.cc
@@ -9,6 +9,7 @@
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
+#include "base/timer/elapsed_timer.h"
 #include "base/timer/timer.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
@@ -57,6 +58,7 @@ void PermissionBlacklistClient::StartCheck(const GURL& request_origin) {
   // empty response if Safe Browsing times out.
   safe_browsing::ThreatMetadata empty_metadata;
   timer_ = base::MakeUnique<base::OneShotTimer>();
+  elapsed_timer_.reset(new base::ElapsedTimer());
   timer_->Start(
       FROM_HERE, base::TimeDelta::FromMilliseconds(timeout_),
       base::Bind(&PermissionBlacklistClient::OnCheckApiBlacklistUrlResult, this,
@@ -69,17 +71,26 @@ void PermissionBlacklistClient::OnCheckApiBlacklistUrlResult(
     const safe_browsing::ThreatMetadata& metadata) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
 
-  if (timer_->IsRunning())
+  base::TimeDelta response_time = elapsed_timer_->Elapsed();
+  SafeBrowsingResponse response = SafeBrowsingResponse::NOT_BLACKLISTED;
+
+  if (timer_->IsRunning()) {
     timer_->Stop();
-  else
+  } else {
     db_manager_->CancelApiCheck(this);
-  timer_.reset(nullptr);
+    response = SafeBrowsingResponse::TIMEOUT;
+  }
 
+  timer_.reset(nullptr);
   bool permission_blocked =
       metadata.api_permissions.find(
           PermissionUtil::ConvertPermissionTypeToSafeBrowsingName(
               permission_type_)) != metadata.api_permissions.end();
+  if (permission_blocked)
+    response = SafeBrowsingResponse::BLACKLISTED;
 
+  // record elapsed time in PermissionUmaUtil

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

This comment is probably not needed.

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

Done.

+  PermissionUmaUtil::RecordSafeBrowsingResponse(response_time, response);
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::Bind(&PermissionBlacklistClient::EvaluateBlacklistResultOnUiThread,
@@ -87,11 +98,11 @@ void PermissionBlacklistClient::OnCheckApiBlacklistUrlResult(
 }
 
 void PermissionBlacklistClient::EvaluateBlacklistResultOnUiThread(
-    bool permission_blocked) {
+    bool response) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
   if (is_active_)
-    callback_.Run(permission_blocked);
+    callback_.Run(response);
   Release();
 }