diff options
| author | scheib@chromium.org <scheib@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-16 01:26:28 +0000 |
|---|---|---|
| committer | scheib@chromium.org <scheib@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-16 01:26:28 +0000 |
| commit | 3b511b52790bdd484b3bb25d3f3a726e449ad4e7 (patch) | |
| tree | 40ec9a87aea8509c1f860423170bed8c58f4bc00 /chrome/browser/background/background_contents_service.cc | |
| parent | 6b9efd9d98221d91e833b38d1a2d9c2fb7312cc7 (diff) | |
| download | chromium_src-3b511b52790bdd484b3bb25d3f3a726e449ad4e7.zip chromium_src-3b511b52790bdd484b3bb25d3f3a726e449ad4e7.tar.gz chromium_src-3b511b52790bdd484b3bb25d3f3a726e449ad4e7.tar.bz2 | |
Unload all apps / extensions immediately when deleting a profile.
Previously apps could remain running with references to profiles that had been deleted by users, but before the browser shut down and profiles were fully removed. Problems included E.g. opening a link in an app would open a tab in the deleted profile.
Relanding patch: ShutdownStartupCycle failed in build [49353], this patch was speculatively reverted in r269383 [49355], but ShutdownStartupCycle failed again in [49362] after the revert had landed.
[49353] http://build.chromium.org/p/chromium.mac/builders/Mac%2010.6%20Tests%20%28dbg%29%281%29/builds/49353
[49355] http://build.chromium.org/p/chromium.mac/builders/Mac%2010.6%20Tests%20%28dbg%29%281%29/builds/49355
[49362] http://build.chromium.org/p/chromium.mac/builders/Mac%2010.6%20Tests%20%28dbg%29%281%29/builds/49362
BUG=368684
TEST=Manual testing as described on http://crbug.com/368684#c1
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269343
Review URL: https://codereview.chromium.org/266343002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270890 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/background/background_contents_service.cc')
| -rw-r--r-- | chrome/browser/background/background_contents_service.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/background/background_contents_service.cc b/chrome/browser/background/background_contents_service.cc index 0d21871..b83decd 100644 --- a/chrome/browser/background/background_contents_service.cc +++ b/chrome/browser/background/background_contents_service.cc @@ -468,7 +468,8 @@ void BackgroundContentsService::Observe( case UnloadedExtensionInfo::REASON_DISABLE: // Fall through. case UnloadedExtensionInfo::REASON_TERMINATE: // Fall through. case UnloadedExtensionInfo::REASON_UNINSTALL: // Fall through. - case UnloadedExtensionInfo::REASON_BLACKLIST: + case UnloadedExtensionInfo::REASON_BLACKLIST: // Fall through. + case UnloadedExtensionInfo::REASON_PROFILE_SHUTDOWN: ShutdownAssociatedBackgroundContents(base::ASCIIToUTF16( content::Details<UnloadedExtensionInfo>(details)-> extension->id())); |