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

Unified Diff: chrome/browser/profiles/profile_impl.cc

Issue 8198007: Remove PrefService::ScheduleSavePersistentPrefs and SavePersistentPrefs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 9 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/prefs/testing_pref_store.cc ('k') | chrome/browser/search_engines/template_url_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/profiles/profile_impl.cc
diff --git a/chrome/browser/profiles/profile_impl.cc b/chrome/browser/profiles/profile_impl.cc
index 8ba1086b46e402bad49fb6dbbcbc0320e0f713aa..e5bda052785505a2081d98a2537ea3a10ed0225a 100644
--- a/chrome/browser/profiles/profile_impl.cc
+++ b/chrome/browser/profiles/profile_impl.cc
@@ -758,8 +758,6 @@ void ProfileImpl::OnPrefsLoaded(bool success) {
prefs_->GetBoolean(prefs::kSessionExitedCleanly);
// Mark the session as open.
prefs_->SetBoolean(prefs::kSessionExitedCleanly, false);
- // Make sure we save to disk that the session has opened.
- prefs_->ScheduleSavePersistentPrefs();
bool extensions_disabled =
prefs_->GetBoolean(prefs::kDisableExtensions) ||
@@ -1291,7 +1289,7 @@ void ProfileImpl::MarkAsCleanShutdown() {
// NOTE: If you change what thread this writes on, be sure and update
// ChromeFrame::EndSession().
- prefs_->SavePersistentPrefs();
+ prefs_->CommitPendingWrite();
}
}
@@ -1533,9 +1531,6 @@ void ProfileImpl::ChangeAppLocale(
if (chromeos::UserManager::Get()->current_user_is_owner())
local_state->SetString(prefs::kOwnerLocale, new_locale);
-
- GetPrefs()->ScheduleSavePersistentPrefs();
- local_state->ScheduleSavePersistentPrefs();
}
void ProfileImpl::OnLogin() {
« no previous file with comments | « chrome/browser/prefs/testing_pref_store.cc ('k') | chrome/browser/search_engines/template_url_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698