UKM Sync Observer

components/metrics_services_manager/metrics_services_manager.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/metrics_services_manager/metrics_services_manager.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "components/metrics/metrics_service.h"
@@ skipping 50 matching lines @@
   GetMetricsServiceClient()->OnPluginLoadingError(plugin_path);
 }
 
 void MetricsServicesManager::OnRendererProcessCrash() {
   GetMetricsServiceClient()->OnRendererProcessCrash();
 }
 
 metrics::MetricsServiceClient*
 MetricsServicesManager::GetMetricsServiceClient() {
   DCHECK(thread_checker_.CalledOnValidThread());
-  if (!metrics_service_client_)
+  if (!metrics_service_client_) {
     metrics_service_client_ = client_->CreateMetricsServiceClient();
+    // base::Unretained is safe since |this| owns the metrics_service_client_.
+    metrics_service_client_->SetUpdateRunningServicesCallback(
+        base::Bind(&MetricsServicesManager::UpdateRunningServices,
+                   base::Unretained(this)));
+  }
   return metrics_service_client_.get();
 }
 
-void MetricsServicesManager::UpdatePermissions(bool may_record,
-                                               bool may_upload) {
+void MetricsServicesManager::UpdatePermissions(bool current_may_record,
+                                               bool current_may_upload) {
   DCHECK(thread_checker_.CalledOnValidThread());
+  // If the user has opted out of metrics, delete local UKM state.
+  if (may_record_ && !current_may_record) {
+    ukm::UkmService* ukm = GetUkmService();
+    if (ukm) {
+      ukm->Purge();
+      ukm->ResetClientId();
+    }
+  }
+
   // Stash the current permissions so that we can update the RapporServiceImpl
   // correctly when the Rappor preference changes.  The metrics recording
   // preference partially determines the initial rappor setting, and also
   // controls whether FINE metrics are sent.
-  may_record_ = may_record;
-  may_upload_ = may_upload;
+  may_record_ = current_may_record;
+  may_upload_ = current_may_upload;
   UpdateRunningServices();
 }
 
 void MetricsServicesManager::UpdateRunningServices() {
   DCHECK(thread_checker_.CalledOnValidThread());
   metrics::MetricsService* metrics = GetMetricsService();
-  ukm::UkmService* ukm = GetUkmService();
 
   if (client_->OnlyDoMetricsRecording()) {
     metrics->StartRecordingForTests();
     GetRapporServiceImpl()->Update(
         rappor::UMA_RAPPOR_GROUP | rappor::SAFEBROWSING_RAPPOR_GROUP, false);
     return;
   }
 
   client_->UpdateRunningServices(may_record_, may_upload_);
 
   if (may_record_) {
     if (!metrics->recording_active())
       metrics->Start();
-
-    if (may_upload_) {
+    if (may_upload_)
       metrics->EnableReporting();
-#if !defined(OFFICIAL_BUILD)
-      // TODO(holte): Make UKM check sync state instead of official build.
-      if (ukm)
-        ukm->EnableReporting();
-#endif
-    } else {
+    else
       metrics->DisableReporting();
-      if (ukm)
-        ukm->DisableReporting();
-    }
   } else {
     metrics->Stop();
   }
 
+  ukm::UkmService* ukm = GetUkmService();

[Alexei Svitkine (slow), 2017/02/04 00:52:23]

Nit: Make a helper function to update ukm state and put this new code there and
call it from here.

[Steven Holte, 2017/02/06 21:57:54]

Done.

+  if (ukm) {
+    if (may_record_ &&
+        metrics_service_client_->IsHistorySyncEnabledOnAllProfiles()) {
+      ukm->EnableRecording();
+      if (may_upload_)
+        ukm->EnableReporting();
+      else
+        ukm->DisableReporting();
+    } else {
+      ukm->DisableRecording();
+      ukm->DisableReporting();
+    }
+  }
+
   int recording_groups = 0;
 #if defined(GOOGLE_CHROME_BUILD)
   if (may_record_)
     recording_groups |= rappor::UMA_RAPPOR_GROUP;
 
   // NOTE: It is safe to use a raw pointer to |this| because this object owns
   // |client_|, and the contract of
   // MetricsServicesManagerClient::IsSafeBrowsingEnabled() states that the
   // callback passed in must not be used beyond the lifetime of the client
   // instance.
   base::Closure on_safe_browsing_update_callback = base::Bind(
       &MetricsServicesManager::UpdateRunningServices, base::Unretained(this));
   if (client_->IsSafeBrowsingEnabled(on_safe_browsing_update_callback))
     recording_groups |= rappor::SAFEBROWSING_RAPPOR_GROUP;
 #endif  // defined(GOOGLE_CHROME_BUILD)
   GetRapporServiceImpl()->Update(recording_groups, may_upload_);
 }
 
 void MetricsServicesManager::UpdateUploadPermissions(bool may_upload) {
   UpdatePermissions(client_->IsMetricsReportingEnabled(), may_upload);
 }
 
 }  // namespace metrics_services_manager