Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5865)

Unified Diff: chrome/browser/safe_browsing/database_manager.cc

Issue 484603006: Add LOCAL_ prefix to non-UMA histogram macros. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments. Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/database_manager.cc
diff --git a/chrome/browser/safe_browsing/database_manager.cc b/chrome/browser/safe_browsing/database_manager.cc
index 84b09428e5104adf40929dee9bc197331563599e..92bf351aff0f0d9be52f0c936a4d607b4e79ba8c 100644
--- a/chrome/browser/safe_browsing/database_manager.cc
+++ b/chrome/browser/safe_browsing/database_manager.cc
@@ -810,7 +810,7 @@ void SafeBrowsingDatabaseManager::DatabaseLoadComplete() {
if (!enabled_)
return;
- HISTOGRAM_COUNTS("SB.QueueDepth", queued_checks_.size());
+ LOCAL_HISTOGRAM_COUNTS("SB.QueueDepth", queued_checks_.size());
if (queued_checks_.empty())
return;
@@ -820,7 +820,8 @@ void SafeBrowsingDatabaseManager::DatabaseLoadComplete() {
while (!queued_checks_.empty()) {
QueuedCheck check = queued_checks_.front();
DCHECK(!check.start.is_null());
- HISTOGRAM_TIMES("SB.QueueDelay", base::TimeTicks::Now() - check.start);
+ LOCAL_HISTOGRAM_TIMES("SB.QueueDelay",
+ base::TimeTicks::Now() - check.start);
// If CheckUrl() determines the URL is safe immediately, it doesn't call the
// client's handler function (because normally it's being directly called by
// the client). Since we're not the client, we have to convey this result.
« no previous file with comments | « chrome/browser/renderer_host/chrome_render_message_filter.cc ('k') | chrome/browser/spellchecker/spellcheck_platform_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698