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

Unified Diff: chrome/browser/chromeos/memory/oom_priority_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/chromeos/memory/oom_priority_manager.cc
diff --git a/chrome/browser/chromeos/memory/oom_priority_manager.cc b/chrome/browser/chromeos/memory/oom_priority_manager.cc
index a6d408ca6a5aafc7b356def7708a7bd2ca2702ed..b948da99e59ed8e81d75ed4a2baa8a76b991357c 100644
--- a/chrome/browser/chromeos/memory/oom_priority_manager.cc
+++ b/chrome/browser/chromeos/memory/oom_priority_manager.cc
@@ -58,7 +58,7 @@ namespace chromeos {
namespace {
// Record a size in megabytes, over a potential interval up to 32 GB.
-#define HISTOGRAM_MEGABYTES(name, sample) \
+#define UMA_HISTOGRAM_MEGABYTES(name, sample) \
UMA_HISTOGRAM_CUSTOM_COUNTS(name, sample, 1, 32768, 50)
// The default interval in seconds after which to adjust the oom_score_adj
@@ -356,7 +356,7 @@ void OomPriorityManager::RecordDiscardStatistics() {
// not have sufficient resolution in the 2-4 GB range and does not properly
// account for graphics memory on ARM. Replace with MemAllocatedMB below.
int mem_anonymous_mb = (memory.active_anon + memory.inactive_anon) / 1024;
- HISTOGRAM_MEGABYTES("Tabs.Discard.MemAnonymousMB", mem_anonymous_mb);
+ UMA_HISTOGRAM_MEGABYTES("Tabs.Discard.MemAnonymousMB", mem_anonymous_mb);
// Record graphics GEM object size in a histogram with 50 MB buckets.
int mem_graphics_gem_mb = 0;
@@ -380,7 +380,7 @@ void OomPriorityManager::RecordDiscardStatistics() {
int mem_available_mb =
(memory.active_file + memory.inactive_file + memory.free) / 1024;
- HISTOGRAM_MEGABYTES("Tabs.Discard.MemAvailableMB", mem_available_mb);
+ UMA_HISTOGRAM_MEGABYTES("Tabs.Discard.MemAvailableMB", mem_available_mb);
}
// Set up to record the next interval.
last_discard_time_ = TimeTicks::Now();
« no previous file with comments | « chrome/browser/chrome_browser_main_mac.mm ('k') | chrome/browser/extensions/external_registry_loader_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698