UKM Sync Observer

components/metrics/metrics_service_client.h

Index: components/metrics/metrics_service_client.h
diff --git a/components/metrics/metrics_service_client.h b/components/metrics/metrics_service_client.h
index 858d1d87db491f533ddcc9611231115b1d41f6ba..f4e829f803e4f2c801bb596989639c1301d1e669 100644
--- a/components/metrics/metrics_service_client.h
+++ b/components/metrics/metrics_service_client.h
@@ -10,7 +10,7 @@
 #include <memory>
 #include <string>
 
-#include "base/callback_forward.h"
+#include "base/callback.h"
 #include "base/strings/string16.h"
 #include "base/time/time.h"
 #include "components/metrics/metrics_reporting_default_state.h"
@@ -33,7 +33,8 @@ class MetricsService;
 // environment.
 class MetricsServiceClient {
  public:
-  virtual ~MetricsServiceClient() {}
+  MetricsServiceClient();
+  virtual ~MetricsServiceClient();
 
   // Returns the MetricsService instance that this client is associated with.
   // With the exception of testing contexts, the returned instance must be valid
@@ -120,6 +121,17 @@ class MetricsServiceClient {
 
   // Returns whether cellular logic is enabled for metrics reporting.
   virtual bool IsUMACellularUploadLogicEnabled();
+
+  // Returns if history sync is enabled on all active profiles.
+  virtual bool IsHistorySyncEnabledOnAllProfiles();
+
+  // Sets the callback to run MetricsServiceManager::UpdateRunningServices.
+  void SetUpdateRunningServicesCallback(const base::Closure& callback);
+
+ protected:

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

This section should be private. Member variables that are protected are
discouraged by style guide.

Maybe just provide a UpdateRunningServices() functions that runs the callback.

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

Done.

+  base::Closure update_running_services_;
+
+  DISALLOW_COPY_AND_ASSIGN(MetricsServiceClient);
 };
 
 }  // namespace metrics