Index: content/browser/storage_partition_impl.cc |
diff --git a/content/browser/storage_partition_impl.cc b/content/browser/storage_partition_impl.cc |
index 5c4270512f9614eafefc00b3d70723b7c9f8f3cd..aa7f8e742ec65ba92b5acfba425b47c9862c56be 100644 |
--- a/content/browser/storage_partition_impl.cc |
+++ b/content/browser/storage_partition_impl.cc |
@@ -22,6 +22,7 @@ |
#include "content/browser/fileapi/browser_file_system_helper.h" |
#include "content/browser/gpu/shader_cache_factory.h" |
#include "content/browser/notifications/platform_notification_context_impl.h" |
+#include "content/browser/renderer_host/render_process_host_impl.h" |
#include "content/common/dom_storage/dom_storage_types.h" |
#include "content/public/browser/browser_context.h" |
#include "content/public/browser/browser_thread.h" |
@@ -401,6 +402,8 @@ StoragePartitionImpl::~StoragePartitionImpl() { |
base::Bind(&storage::DatabaseTracker::Shutdown, GetDatabaseTracker())); |
} |
+ RenderProcessHostImpl::OnStoragePartitionShutdown(this); |
Charlie Reis
2017/06/25 23:48:28
Hmm. This feels a bit like a layering violation /
mattcary
2017/06/26 14:45:07
Not that I know of. The idea here is that if the p
Charlie Reis
2017/06/26 21:22:50
If the profile is deleted while it still has RPHs,
mattcary
2017/06/28 13:14:38
As far as I can tell, deleting a browser context w
Charlie Reis
2017/06/28 22:31:35
That's interesting about BrowserContext::NotifyWil
mattcary
2017/06/29 12:57:09
Actually, it's already done for us: NotifyWillBeDe
Charlie Reis
2017/06/29 21:09:16
Oh, that's pretty indirect! I'd worry about that
mattcary
2017/06/30 14:29:46
Added the comment of (2) and (3).
|
+ |
if (GetFileSystemContext()) |
GetFileSystemContext()->Shutdown(); |