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

Unified Diff: chrome/browser/memory/memory_kills_histogram.h

Issue 2527973003: Consolidate code monitoring low memory kills and OOM kills to MemoryKillsMonitor on ChromeOS. (Closed)
Patch Set: move unittest to chromeos only build rule Created 4 years 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/memory/memory_kills_histogram.h
diff --git a/chrome/browser/memory/memory_kills_histogram.h b/chrome/browser/memory/memory_kills_histogram.h
new file mode 100644
index 0000000000000000000000000000000000000000..67f1641c3b35d7ceb8713f2247d4b97c7b8ffab1
--- /dev/null
+++ b/chrome/browser/memory/memory_kills_histogram.h
@@ -0,0 +1,25 @@
+// Copyright 2016 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.
+
+#ifndef CHROME_BROWSER_MEMORY_MEMORY_KILLS_HISTOGRAM_H_
+#define CHROME_BROWSER_MEMORY_MEMORY_KILLS_HISTOGRAM_H_
+
+#include "base/metrics/histogram.h"
+#include "base/time/time.h"
+
+namespace memory {
+
+constexpr base::TimeDelta kMaxMemoryKillTimeDelta =
+ base::TimeDelta::FromSeconds(30);
+
+} // namespace memory
+
+// Use this macro to report elapsed time since last Memory kill event.
+// Must be a macro as the underlying HISTOGRAM macro creates static variables.
+#define UMA_HISTOGRAM_MEMORY_KILL_TIME_INTERVAL(name, sample) \
+ UMA_HISTOGRAM_CUSTOM_TIMES( \
+ name, sample, base::TimeDelta::FromMilliseconds(1), \
+ ::memory::kMaxMemoryKillTimeDelta, 50)
+
+#endif // CHROME_BROWSER_MEMORY_MEMORY_KILLS_HISTOGRAM_H_
« no previous file with comments | « chrome/browser/chromeos/chrome_browser_main_chromeos.cc ('k') | chrome/browser/memory/memory_kills_monitor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698