Add metrics for Safe Browsing blacklist response.

chrome/browser/permissions/permission_blacklist_client.h

Index: chrome/browser/permissions/permission_blacklist_client.h
diff --git a/chrome/browser/permissions/permission_blacklist_client.h b/chrome/browser/permissions/permission_blacklist_client.h
index 239614362639a43dbec14e4be4c2a5fce6400df3..3dc764180b672ba3b39198a7968ebd9fcb0e6072 100644
--- a/chrome/browser/permissions/permission_blacklist_client.h
+++ b/chrome/browser/permissions/permission_blacklist_client.h
@@ -5,9 +5,11 @@
 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_BLACKLIST_CLIENT_H_
 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_BLACKLIST_CLIENT_H_
 
+#include <memory>
+
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
-#include "chrome/browser/permissions/permission_util.h"
+#include "chrome/browser/permissions/permission_uma_util.h"

[dominickn, 2017/02/07 05:36:54]

Move this include to the cc file?

[meredithl, 2017/02/07 06:27:29]

Done.

 #include "components/safe_browsing_db/database_manager.h"
 #include "content/public/browser/permission_type.h"
 #include "content/public/browser/web_contents_observer.h"
@@ -20,6 +22,7 @@ class WebContents;
 
 namespace base {
 class OneShotTimer;
+class ElapsedTimer;
 }
 
 // The client used when checking whether a permission has been blacklisted by
@@ -62,7 +65,7 @@ class PermissionBlacklistClient
       const GURL& url,
       const safe_browsing::ThreatMetadata& metadata) override;
 
-  void EvaluateBlacklistResultOnUiThread(bool permission_blocked);
+  void EvaluateBlacklistResultOnUiThread(bool response);
 
   // WebContentsObserver implementation. Sets a flag so that when the database
   // manager returns with a result, it won't attempt to run the callback with a
@@ -78,6 +81,7 @@ class PermissionBlacklistClient
   // Timer to abort the Safe Browsing check if it takes too long. Created and
   // used on the IO Thread.
   std::unique_ptr<base::OneShotTimer> timer_;
+  std::unique_ptr<base::ElapsedTimer> elapsed_timer_;
   int timeout_;
 
   // True if |callback_| should be invoked, if web_contents() is destroyed, this