diff options
| author | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-06 14:27:50 +0000 |
|---|---|---|
| committer | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-06 14:27:50 +0000 |
| commit | a2c04390a1e377fc82be7ec58d5f570ee70487fd (patch) | |
| tree | 8cfed57479c72e8f0a7072e5104821c95a3e6c3c /chrome/browser/browser_shutdown.cc | |
| parent | dcd93f931548d928d2ede12da6e3395a9e12a2de (diff) | |
| download | chromium_src-a2c04390a1e377fc82be7ec58d5f570ee70487fd.zip chromium_src-a2c04390a1e377fc82be7ec58d5f570ee70487fd.tar.gz chromium_src-a2c04390a1e377fc82be7ec58d5f570ee70487fd.tar.bz2 | |
Make profile deletion really remove stale profile directory
Before this CL, after deleting a profile and restarting the browser - the old profile directory would persist and still contained some files. This happened because the code to physically delete the profile directory was running before the profile object had been deleted and some files were being written back out to disk recreating the directory.
This patch moves the "nuking" code late in the shutdown process to fix the order of these events.
BUG=95079
TEST=See bug
Review URL: http://codereview.chromium.org/8133002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104297 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_shutdown.cc')
| -rw-r--r-- | chrome/browser/browser_shutdown.cc | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/chrome/browser/browser_shutdown.cc b/chrome/browser/browser_shutdown.cc index dfcbfb8..2170b71 100644 --- a/chrome/browser/browser_shutdown.cc +++ b/chrome/browser/browser_shutdown.cc @@ -161,6 +161,11 @@ void Shutdown() { // before calling UninstallJankometer(). delete g_browser_process; g_browser_process = NULL; + + // crbug.com/95079 - This needs to happen after the browser process object + // goes away. + ProfileManager::NukeDeletedProfilesFromDisk(); + #if defined(OS_CHROMEOS) chromeos::BootTimesLoader::Get()->AddLogoutTimeMarker("BrowserDeleted", true); |