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

Unified Diff: chrome/browser/metrics/metrics_services_manager.cc

Issue 291163006: Create GPUMetricsProvider. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Response to review Created 6 years, 7 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/metrics_services_manager.cc
diff --git a/chrome/browser/metrics/metrics_services_manager.cc b/chrome/browser/metrics/metrics_services_manager.cc
index 002f18383235aebe6ef0c5202704345f570b54c1..24c3b7a4c9cf95fc0f57b33d6a74bd662b2663b6 100644
--- a/chrome/browser/metrics/metrics_services_manager.cc
+++ b/chrome/browser/metrics/metrics_services_manager.cc
@@ -4,6 +4,7 @@
#include "chrome/browser/metrics/metrics_services_manager.h"
+#include "chrome/browser/metrics/gpu_metrics_provider.h"
#include "chrome/browser/metrics/metrics_service.h"
#include "chrome/browser/metrics/metrics_state_manager.h"
#include "chrome/browser/metrics/variations/variations_service.h"
@@ -19,8 +20,14 @@ MetricsServicesManager::~MetricsServicesManager() {
MetricsService* MetricsServicesManager::GetMetricsService() {
DCHECK(thread_checker_.CalledOnValidThread());
- if (!metrics_service_)
+ if (!metrics_service_) {
metrics_service_.reset(new MetricsService(GetMetricsStateManager()));
+
+ // TODO(asvitkine): Move the registration of the metrics providers into
+ // ChromeMetricsServiceClient. crbug.com/375248
+ metrics_service_->RegisterDataProvider(
+ scoped_ptr<metrics::MetricsDataProvider>(new GPUMetricsProvider()));
Ilya Sherman 2014/05/20 14:40:27 nit: "MetricsDataProvider" -> "MetricsProvider"
blundell 2014/05/20 14:59:11 I'm going to rebase this once Alexei's patch lands
+ }
return metrics_service_.get();
}

Powered by Google App Engine
This is Rietveld 408576698