Move EntropyProvider Source From MetricsService to MetricsServicesManager

chrome/browser/metrics/chrome_metrics_services_manager_client.h

 // Copyright 2015 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.
 
 #ifndef CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_
 #define CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_
 
 #include <memory>
 
 #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"
 
 class PrefService;
 
 namespace metrics {
 class EnabledStateProvider;
 class MetricsStateManager;
 }
@@ skipping 35 matching lines @@
   // ChromeMetricsServiceAccessor through ChromeMetricsServicesManagerClient's
   // friendship.
   class ChromeEnabledStateProvider;
 
   // metrics_services_manager::MetricsServicesManagerClient:
   std::unique_ptr<rappor::RapporService> CreateRapporService() override;
   std::unique_ptr<variations::VariationsService> CreateVariationsService()
       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;
   bool OnlyDoMetricsRecording() override;
 
 #if defined(OS_WIN)
   // On Windows, the client controls whether Crashpad can upload crash reports.
   void UpdateRunningServices(bool may_record, bool may_upload) override;
 #endif  // defined(OS_WIN)
 
@@ skipping 15 matching lines @@
   PrefService* local_state_;
 
   // Subscription to SafeBrowsing service state changes.
   std::unique_ptr<safe_browsing::SafeBrowsingService::StateSubscription>
       sb_state_subscription_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeMetricsServicesManagerClient);
 };
 
 #endif  // CHROME_BROWSER_METRICS_CHROME_METRICS_SERVICES_MANAGER_CLIENT_H_