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

Unified Diff: chrome/browser/metrics/chromeos_metrics_provider.h

Issue 1681263003: metrics: Add leak detector controller in Chrome OS metrics system (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove frame ptr flag, add thread checker, add #define flag locally (gn build only) Created 4 years, 10 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/metrics/chromeos_metrics_provider.h
diff --git a/chrome/browser/metrics/chromeos_metrics_provider.h b/chrome/browser/metrics/chromeos_metrics_provider.h
index 6b1c90040b0f554e74354a4eec921792c8d40d7c..e8391b720e51fd62252c220b7d6b2085a234ac19 100644
--- a/chrome/browser/metrics/chromeos_metrics_provider.h
+++ b/chrome/browser/metrics/chromeos_metrics_provider.h
@@ -9,7 +9,9 @@
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
+#include "chrome/browser/metrics/leak_detector_controller.h"
#include "chrome/browser/metrics/perf/perf_provider_chromeos.h"
+#include "components/metrics/metrics_flags.h"
#include "components/metrics/metrics_provider.h"
namespace device {
@@ -80,8 +82,14 @@ class ChromeOSMetricsProvider : public metrics::MetricsProvider {
// Record the device enrollment status.
void RecordEnrollmentStatus();
+ // For collecting systemwide perf data.
metrics::PerfProvider perf_provider_;
+#if defined(ENABLE_LEAK_DETECTOR)
+ // For runtime memory leak detection
Alexei Svitkine (slow) 2016/02/18 21:09:19 Nit: Full sentences.
Simon Que 2016/02/19 02:13:02 Done.
+ metrics::LeakDetectorController leak_detector_controller_;
+#endif
+
// Bluetooth Adapter instance for collecting information about paired devices.
scoped_refptr<device::BluetoothAdapter> adapter_;

Powered by Google App Engine
This is Rietveld 408576698