diff options
| author | hashimoto <hashimoto@chromium.org> | 2015-04-21 18:46:07 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-04-22 01:46:21 +0000 |
| commit | f51c15827319393fbdcd9538586fa42b51ee2d6a (patch) | |
| tree | e859e0bd0418d3cb9754626b8220ac5424430bae /extensions | |
| parent | f41d0865f34a26fc68221c8d2020a65779fc7361 (diff) | |
| download | chromium_src-f51c15827319393fbdcd9538586fa42b51ee2d6a.zip chromium_src-f51c15827319393fbdcd9538586fa42b51ee2d6a.tar.gz chromium_src-f51c15827319393fbdcd9538586fa42b51ee2d6a.tar.bz2 | |
Do not delete the temporary directory in SandboxedUnpacker's dtor
In practice, this undeleted directory will be deleted by ExtensionGarbageCollector in chrome/browser/extensions.
BUG=477644
Review URL: https://codereview.chromium.org/1061493003
Cr-Commit-Position: refs/heads/master@{#326215}
Diffstat (limited to 'extensions')
| -rw-r--r-- | extensions/browser/sandboxed_unpacker.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/extensions/browser/sandboxed_unpacker.cc b/extensions/browser/sandboxed_unpacker.cc index 29d559a..18823cb 100644 --- a/extensions/browser/sandboxed_unpacker.cc +++ b/extensions/browser/sandboxed_unpacker.cc @@ -322,6 +322,9 @@ void SandboxedUnpacker::Start() { } SandboxedUnpacker::~SandboxedUnpacker() { + // To avoid blocking shutdown, don't delete temporary directory here if it + // hasn't been cleaned up or passed on to another owner yet. + temp_dir_.Take(); } bool SandboxedUnpacker::OnMessageReceived(const IPC::Message& message) { |