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

Unified Diff: chrome/browser/browser_process_impl.cc

Issue 11570009: Split PrefService into PrefService, PrefServiceSimple and PrefServiceSyncable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge to head again, previous had unrelated broken win_rel test. Created 8 years 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/browser_process_impl.h ('k') | chrome/browser/browser_shutdown.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index aa144cabe16644b0e21b0623fdc4123b50d5920d..3a72d680bd5ff1a8038264427e76c27f01049b84 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -49,7 +49,7 @@
#include "chrome/browser/plugins/plugin_finder.h"
#include "chrome/browser/policy/policy_service.h"
#include "chrome/browser/prefs/browser_prefs.h"
-#include "chrome/browser/prefs/chrome_pref_service_builder.h"
+#include "chrome/browser/prefs/chrome_pref_service_factory.h"
#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/prerender/prerender_tracker.h"
#include "chrome/browser/printing/background_printing_manager.h"
@@ -400,7 +400,10 @@ ProfileManager* BrowserProcessImpl::profile_manager() {
return profile_manager_.get();
}
-PrefService* BrowserProcessImpl::local_state() {
+// TODO(joi): Switch to returning just PrefService, since those
+// calling this function shouldn't be doing ad-hoc registration, that
+// happens earlier in browser_prefs::RegisterLocalState.
+PrefServiceSimple* BrowserProcessImpl::local_state() {
DCHECK(CalledOnValidThread());
if (!created_local_state_)
CreateLocalState();
@@ -735,10 +738,10 @@ void BrowserProcessImpl::CreateLocalState() {
FilePath local_state_path;
CHECK(PathService::Get(chrome::FILE_LOCAL_STATE, &local_state_path));
local_state_.reset(
- ChromePrefServiceBuilder().CreateChromePrefs(local_state_path,
- local_state_task_runner_,
- policy_service(),
- NULL, false));
+ chrome_prefs::CreateLocalState(local_state_path,
+ local_state_task_runner_,
+ policy_service(),
+ NULL, false));
// Initialize the prefs of the local state.
chrome::RegisterLocalState(local_state_.get());
« no previous file with comments | « chrome/browser/browser_process_impl.h ('k') | chrome/browser/browser_shutdown.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698