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

Unified Diff: components/metrics_services_manager/metrics_services_manager.cc

Issue 2861563002: Add an override flag to enable metrics reporting. (Closed)
Patch Set: Rebase 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..15bfccec89c44272204825cc6296b207a9298571 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_->IsMetricsReportingForceEnabled() ||
metrics_service_client_->IsHistorySyncEnabledOnAllProfiles();
if (may_record_ && sync_enabled) {
ukm->EnableRecording();
@@ -146,7 +147,9 @@ void MetricsServicesManager::UpdateUkmService() {
}
void MetricsServicesManager::UpdateUploadPermissions(bool may_upload) {
- UpdatePermissions(client_->IsMetricsReportingEnabled(), may_upload);
+ UpdatePermissions((client_->IsMetricsReportingForceEnabled() ||
+ client_->IsMetricsReportingEnabled()),
+ client_->IsMetricsReportingForceEnabled() || may_upload);
}
} // namespace metrics_services_manager
« no previous file with comments | « components/metrics_services_manager/BUILD.gn ('k') | components/metrics_services_manager/metrics_services_manager_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698