Move EntropyProvider Source From MetricsService to MetricsServicesManager

chrome/browser/metrics/chrome_metrics_services_manager_client.h

Index: chrome/browser/metrics/chrome_metrics_services_manager_client.h
diff --git a/chrome/browser/metrics/chrome_metrics_services_manager_client.h b/chrome/browser/metrics/chrome_metrics_services_manager_client.h
index 0a4ffb17dd10fa9f9592f8d772532ee0c30ddd5e..fe3863fd60c68bd19e4b46a38433ea22e4fdf538 100644
--- a/chrome/browser/metrics/chrome_metrics_services_manager_client.h
+++ b/chrome/browser/metrics/chrome_metrics_services_manager_client.h
@@ -9,6 +9,7 @@
 
 #include "base/feature_list.h"
 #include "base/macros.h"
+#include "base/metrics/field_trial.h"
 #include "base/threading/thread_checker.h"
 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
 #include "components/metrics_services_manager/metrics_services_manager_client.h"
@@ -64,6 +65,8 @@ class ChromeMetricsServicesManagerClient
       override;
   std::unique_ptr<metrics::MetricsServiceClient> CreateMetricsServiceClient()
       override;
+  std::unique_ptr<const base::FieldTrial::EntropyProvider>
+  CreateEntropyProvider() override;

[brettw, 2016/09/14 22:22:03]

I think the function name should be indented 4 spaces here. Now it looks like
two functions.

[robliao, 2016/09/15 14:31:18]

Referring to 
https://codereview.chromium.org/2333283005/diff/60001/components/metrics_serv...
on this

   net::URLRequestContextGetter* GetURLRequestContext() override;
   bool IsSafeBrowsingEnabled(const base::Closure& on_update_callback) override;
   bool IsMetricsReportingEnabled() override;