Add histograms to measure new SafeBrowsing performance.
Review URL: http://codereview.chromium.org/10712
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5423 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/safe_browsing/safe_browsing_service.cc b/chrome/browser/safe_browsing/safe_browsing_service.cc
index c09806d7..a2c62a2b 100644
--- a/chrome/browser/safe_browsing/safe_browsing_service.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_service.cc
@@ -202,9 +202,13 @@
std::string list;
std::vector<SBPrefix> prefix_hits;
std::vector<SBFullHashResult> full_hits;
+ base::Time check_start = base::Time::Now();
bool prefix_match = database_->ContainsUrl(url, &list, &prefix_hits,
&full_hits,
protocol_manager_->last_update());
+
+ UMA_HISTOGRAM_TIMES(L"SB2.FilterCheck", base::Time::Now() - check_start);
+
if (!prefix_match)
return true; // URL is okay.
@@ -406,8 +410,12 @@
DCHECK(enabled_);
+ if (new_safe_browsing_)
+ UMA_HISTOGRAM_LONG_TIMES(L"SB2.Network", Time::Now() - check->start);
+ else
+ UMA_HISTOGRAM_LONG_TIMES(L"SB.Network", Time::Now() - check->start);
+
std::vector<SBPrefix> prefixes = check->prefix_hits;
- UMA_HISTOGRAM_LONG_TIMES(L"SB.Network", Time::Now() - check->start);
OnHandleGetHashResults(check, full_hashes); // 'check' is deleted here.
if (can_cache) {
