Added tests for storing and loading seed data to and from prefs.

chrome/browser/metrics/variations_service.h

 // Copyright (c) 2012 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_VARIATIONS_SERVICE_H_
 #define CHROME_BROWSER_METRICS_VARIATIONS_SERVICE_H_
 #pragma once
 
 #include <string>
 
@@ skipping 40 matching lines @@
   static void RegisterPrefs(PrefService* prefs);
 
  private:
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyChannel);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyLocale);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyPlatform);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyVersion);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyVersionWildcards);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyStartDate);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, IsStudyExpired);
+  FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, LoadSeed);
+  FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, StoreSeed);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, ValidateStudy);
 
   // Store the given seed data to the given local prefs. Note that |seed_data|
   // is assumed to be the raw serialized protobuf data stored in a string. It
   // will be Base64Encoded for storage. If the string is invalid or the encoding
-  // fails, the |local_prefs| is left as is.
-  void StoreSeedData(const std::string& seed_data,
+  // fails, the |local_prefs| is left as is and the function returns false.
+  bool StoreSeedData(const std::string& seed_data,
                      const base::Time& seed_date,
                      PrefService* local_prefs);
 
   // Returns whether |study| should be disabled according to its restriction
   // parameters. Uses |version_info| for min / max version checks and
   // |reference_date| for the start date check.
   static bool ShouldAddStudy(const Study& study,
                              const chrome::VersionInfo& version_info,
                              const base::Time& reference_date);
 
@@ skipping 44 matching lines @@
   // The URL to use for querying the variations server.
   GURL variations_server_url_;
 
   // Keep a weak pointer generator so we can bind delayed calls to the server.
   base::WeakPtrFactory<VariationsService> weak_factory_;
 };
 
 }  // namespace chrome_variations
 
 #endif  // CHROME_BROWSER_METRICS_VARIATIONS_SERVICE_H_