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

Unified Diff: ios/chrome/browser/metrics/ios_chrome_metrics_service_client.mm

Issue 2567263003: Basic UkmService implementation (Closed)
Patch Set: Fix small bugs Created 3 years, 11 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: ios/chrome/browser/metrics/ios_chrome_metrics_service_client.mm
diff --git a/ios/chrome/browser/metrics/ios_chrome_metrics_service_client.mm b/ios/chrome/browser/metrics/ios_chrome_metrics_service_client.mm
index 43fee09cca472b15b7f37dd14a2cf8a56941fd5d..d1d5c45576e8523cfef5f8d4f609642706e84ef5 100644
--- a/ios/chrome/browser/metrics/ios_chrome_metrics_service_client.mm
+++ b/ios/chrome/browser/metrics/ios_chrome_metrics_service_client.mm
@@ -41,6 +41,7 @@
#include "components/signin/core/browser/signin_status_metrics_provider.h"
#include "components/sync/device_info/device_count_metrics_provider.h"
#include "components/translate/core/browser/translate_ranker_metrics_provider.h"
+#include "components/ukm/ukm_service.h"
#include "components/variations/variations_associated_data.h"
#include "components/version_info/version_info.h"
#include "ios/chrome/browser/application_context.h"
@@ -91,12 +92,17 @@ void IOSChromeMetricsServiceClient::RegisterPrefs(
metrics::MetricsService::RegisterPrefs(registry);
metrics::StabilityMetricsHelper::RegisterPrefs(registry);
metrics::RegisterMetricsReportingStatePrefs(registry);
+ ukm::UkmService::RegisterPrefs(registry);
}
metrics::MetricsService* IOSChromeMetricsServiceClient::GetMetricsService() {
return metrics_service_.get();
}
+ukm::UkmService* IOSChromeMetricsServiceClient::GetUkmService() {
+ return ukm_service_.get();
+}
+
void IOSChromeMetricsServiceClient::SetMetricsClientId(
const std::string& client_id) {
crash_keys::SetMetricsClientIdFromGUID(client_id);
@@ -182,8 +188,12 @@ void IOSChromeMetricsServiceClient::WebStateDidStopLoading(
}
void IOSChromeMetricsServiceClient::Initialize() {
+ PrefService* local_state = GetApplicationContext()->GetLocalState();
metrics_service_ = base::MakeUnique<metrics::MetricsService>(
- metrics_state_manager_, this, GetApplicationContext()->GetLocalState());
+ metrics_state_manager_, this, local_state);
+
+ if (base::FeatureList::IsEnabled(ukm::kUkmFeature))
+ ukm_service_.reset(new ukm::UkmService(local_state, this));
// Register metrics providers.
metrics_service_->RegisterMetricsProvider(

Powered by Google App Engine
This is Rietveld 408576698