Prepare to abstract PersistedData by making it part of the configurator.

components/update_client/test_configurator.h

Index: components/update_client/test_configurator.h
diff --git a/components/update_client/test_configurator.h b/components/update_client/test_configurator.h
index 1f26ec051d223e159c6e7d3a4f311b5c342357d7..9a3fd557762d12326a68a899825521b28231e6cf 100644
--- a/components/update_client/test_configurator.h
+++ b/components/update_client/test_configurator.h
@@ -82,8 +82,8 @@ class TestConfigurator : public Configurator {
   bool EnabledCupSigning() const override;
   scoped_refptr<base::SequencedTaskRunner> GetSequencedTaskRunner()
       const override;
-  PrefService* GetPrefService() const override;
   bool IsPerUserInstall() const override;
+  std::unique_ptr<PersistedData> CreateMetadata() const override;
 
   void SetBrand(const std::string& brand);
   void SetOnDemandTime(int seconds);
@@ -93,6 +93,7 @@ class TestConfigurator : public Configurator {
   void SetEnabledComponentUpdates(bool enabled_component_updates);
   void SetUpdateCheckUrl(const GURL& url);
   void SetPingUrl(const GURL& url);
+  void SetPrefService(PrefService* pref);
 
  private:
   friend class base::RefCountedThreadSafe<TestConfigurator>;
@@ -109,6 +110,7 @@ class TestConfigurator : public Configurator {
   bool enabled_component_updates_;
   GURL update_check_url_;
   GURL ping_url_;
+  PrefService* pref_;

[Devlin, 2017/06/05 16:13:38]

nit: maybe s/pref_/pref_service_?  A pref is something returned by the pref
service.

[Minh X. Nguyen, 2017/06/05 18:16:38]

Done.

 
   scoped_refptr<net::TestURLRequestContextGetter> context_;