Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1395)

Unified Diff: chrome/browser/metrics/variations/variations_service.h

Issue 136003008: [Variations] Read the policy from profile prefs if on Android (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: DCHECK Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/metrics/variations/variations_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/variations/variations_service.h
diff --git a/chrome/browser/metrics/variations/variations_service.h b/chrome/browser/metrics/variations/variations_service.h
index 2ad70265263a69d70d7e0fe23fbbd4ab14478ade..0f8db1aa2b2761b5e593c8509cdf37857c84ace2 100644
--- a/chrome/browser/metrics/variations/variations_service.h
+++ b/chrome/browser/metrics/variations/variations_service.h
@@ -26,6 +26,10 @@
class PrefService;
class PrefRegistrySimple;
+namespace user_prefs {
+class PrefRegistrySyncable;
+}
+
namespace chrome_variations {
class VariationsSeed;
@@ -67,9 +71,19 @@ class VariationsService
// Register Variations related prefs in Local State.
static void RegisterPrefs(PrefRegistrySimple* registry);
+ // Register Variations related prefs in the Profile prefs.
+ static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);
+
// Factory method for creating a VariationsService.
static VariationsService* Create(PrefService* local_state);
+ // Set the PrefService responsible for getting policy-related preferences,
+ // such as the restrict parameter.
+ void set_policy_pref_service(PrefService* service) {
+ DCHECK(service != NULL);
gab 2014/02/11 20:31:46 nit: DCHECK(service); is sufficient
Mathieu 2014/02/11 20:38:03 Done.
+ policy_pref_service_ = service;
+ }
+
protected:
// Starts the fetching process once, where |OnURLFetchComplete| is called with
// the response.
@@ -110,6 +124,10 @@ class VariationsService
// The pref service used to store persist the variations seed.
PrefService* local_state_;
+ // Used to obtain policy-related preferences. Depending on the platform, will
+ // either be Local State or Profile prefs.
+ PrefService* policy_pref_service_;
+
VariationsSeedStore seed_store_;
// Contains the scheduler instance that handles timing for requests to the
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/metrics/variations/variations_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698