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

Unified Diff: components/sync_preferences/pref_service_syncable_factory.h

Issue 2740493002: Pref service: create service at browser startup (Closed)
Patch Set: Merge Created 3 years, 9 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 | « components/sync_preferences/DEPS ('k') | components/sync_preferences/pref_service_syncable_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync_preferences/pref_service_syncable_factory.h
diff --git a/components/sync_preferences/pref_service_syncable_factory.h b/components/sync_preferences/pref_service_syncable_factory.h
index 9a8994b6c6afe5a833e832823c56cea2c7e2a705..3b9bb3944211379f9e27b7b6a78217c7eddbd795 100644
--- a/components/sync_preferences/pref_service_syncable_factory.h
+++ b/components/sync_preferences/pref_service_syncable_factory.h
@@ -15,6 +15,10 @@ class BrowserPolicyConnector;
class PolicyService;
}
+namespace service_manager {
+class Connector;
+}
+
namespace user_prefs {
class PrefRegistrySyncable;
}
@@ -42,8 +46,11 @@ class PrefServiceSyncableFactory : public PrefServiceFactory {
void SetPrefModelAssociatorClient(
PrefModelAssociatorClient* pref_model_associator_client);
+ // |connector| might be null during test or if we're not using the Mojo pref
+ // |service.
std::unique_ptr<PrefServiceSyncable> CreateSyncable(
- user_prefs::PrefRegistrySyncable* registry);
+ user_prefs::PrefRegistrySyncable* registry,
+ service_manager::Connector* connector = nullptr);
private:
PrefModelAssociatorClient* pref_model_associator_client_;
« no previous file with comments | « components/sync_preferences/DEPS ('k') | components/sync_preferences/pref_service_syncable_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698