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

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

Issue 944433002: Revamp the MigrateBrowserPrefs and MigrateUserPrefs code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: readd kbrowserwindowplacement to ui_prefs.cc Created 5 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
Index: chrome/browser/profiles/profile_impl.cc
diff --git a/chrome/browser/profiles/profile_impl.cc b/chrome/browser/profiles/profile_impl.cc
index 9b7389fc542cb6ec4b5e9f6408f8e529c20c3b50..05bd533d4a6ece678fe3d1b9d5096a95360bb7e8 100644
--- a/chrome/browser/profiles/profile_impl.cc
+++ b/chrome/browser/profiles/profile_impl.cc
@@ -851,11 +851,10 @@ void ProfileImpl::OnPrefsLoaded(bool success) {
return;
}
- // TODO(mirandac): remove migration code after 6 months (crbug.com/69995).
+ // Migrate obsolete prefs.
if (g_browser_process->local_state())
- chrome::MigrateBrowserPrefs(this, g_browser_process->local_state());
- // TODO(ivankr): remove cleanup code eventually (crbug.com/165672).
- chrome::MigrateUserPrefs(this);
+ chrome::MigrateObsoleteBrowserPrefs(this, g_browser_process->local_state());
+ chrome::MigrateObsoleteProfilePrefs(this);
// |kSessionExitType| was added after |kSessionExitedCleanly|. If the pref
// value is empty fallback to checking for |kSessionExitedCleanly|.

Powered by Google App Engine
This is Rietveld 408576698