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

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

Issue 2905903002: Delete the PreferenceMACs on profile deletion. (Closed)
Patch Set: Created 3 years, 7 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 | « no previous file | components/prefs/in_memory_pref_store.h » ('j') | components/prefs/pref_service.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/profiles/profile_manager.cc
diff --git a/chrome/browser/profiles/profile_manager.cc b/chrome/browser/profiles/profile_manager.cc
index 72e0f2ae5dd178efc7c9b648f07919765a293afa..5e6615e5202861bb11d967b92453b7362b73dd87 100644
--- a/chrome/browser/profiles/profile_manager.cc
+++ b/chrome/browser/profiles/profile_manager.cc
@@ -1491,6 +1491,9 @@ void ProfileManager::FinishDeletingProfile(
ProfileSyncServiceFactory::GetInstance()
->GetForProfile(profile)
->RequestStop(browser_sync::ProfileSyncService::CLEAR_DATA);
+
+ // Clean-up pref data that won't be cleaned up by NukeProfileFromDisk.
+ profile->GetPrefs()->CleanupForProfileDeletion();
gab 2017/06/05 17:28:57 Why is this inside if (ProfileSyncServiceFactory::
proberge 2017/06/05 21:22:11 My bad, I thought I put it under if (profile) {}.
}
ProfileAttributesEntry* entry;
« no previous file with comments | « no previous file | components/prefs/in_memory_pref_store.h » ('j') | components/prefs/pref_service.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698