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

Unified Diff: components/metrics_services_manager/metrics_services_manager.cc

Issue 2861563002: Add an override flag to enable metrics reporting. (Closed)
Patch Set: Rename flags & functions Created 3 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: components/metrics_services_manager/metrics_services_manager.cc
diff --git a/components/metrics_services_manager/metrics_services_manager.cc b/components/metrics_services_manager/metrics_services_manager.cc
index 5987635fbacfbf3d479b89ff07eeb33c4e116827..61d41327775c3c071538636354b3f45c9c012d32 100644
--- a/components/metrics_services_manager/metrics_services_manager.cc
+++ b/components/metrics_services_manager/metrics_services_manager.cc
@@ -132,6 +132,7 @@ void MetricsServicesManager::UpdateUkmService() {
if (!ukm)
return;
bool sync_enabled =
+ client_->IsMetricsReportingEnabledForTesting() ||
metrics_service_client_->IsHistorySyncEnabledOnAllProfiles();
if (may_record_ && sync_enabled) {
ukm->EnableRecording();
@@ -146,7 +147,10 @@ void MetricsServicesManager::UpdateUkmService() {
}
void MetricsServicesManager::UpdateUploadPermissions(bool may_upload) {
- UpdatePermissions(client_->IsMetricsReportingEnabled(), may_upload);
+ UpdatePermissions(
+ (client_->IsMetricsReportingEnabledForTesting() ||
+ client_->IsMetricsReportingEnabled()),
+ client_->IsMetricsReportingEnabledForTesting() || may_upload);
}
} // namespace metrics_services_manager

Powered by Google App Engine
This is Rietveld 408576698