Add metrics for Safe Browsing blacklist response.

chrome/browser/permissions/permission_blacklist_client.cc

 // Copyright 2017 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.
 
 #include "chrome/browser/permissions/permission_blacklist_client.h"
 
 #include <set>
 #include <string>
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
+#include "base/timer/elapsed_timer.h"
 #include "base/timer/timer.h"
+#include "chrome/browser/permissions/permission_uma_util.h"
+#include "chrome/browser/permissions/permission_util.h"

[raymes, 2017/02/08 02:08:00]

nit: is this needed?

[meredithl, 2017/02/08 02:40:32]

It is used to convert between the PermissionType to the Safe Browsing string.

 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
 #include "url/gurl.h"
 
 // static
 void PermissionBlacklistClient::CheckSafeBrowsingBlacklist(
     scoped_refptr<safe_browsing::SafeBrowsingDatabaseManager> db_manager,
     content::PermissionType permission_type,
     const GURL& request_origin,
     content::WebContents* web_contents,
@@ skipping 27 matching lines @@
 
 PermissionBlacklistClient::~PermissionBlacklistClient() {}
 
 void PermissionBlacklistClient::StartCheck(const GURL& request_origin) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
 
   // Start the timer to interrupt into the client callback method with an
   // 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,
                  request_origin, empty_metadata));
   db_manager_->CheckApiBlacklistUrl(request_origin, this);
 }
 
 void PermissionBlacklistClient::OnCheckApiBlacklistUrlResult(
     const GURL& url,
     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);
+    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;
 
+  PermissionUmaUtil::RecordSafeBrowsingResponse(response_time, response);
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::Bind(&PermissionBlacklistClient::EvaluateBlacklistResultOnUiThread,
                  this, permission_blocked));
 }
 
 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();
 }
 
 void PermissionBlacklistClient::WebContentsDestroyed() {
   is_active_ = false;
   Observe(nullptr);
 }