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

Unified Diff: content/browser/storage_partition_impl.cc

Issue 1014703007: The platform notification context should observe the Service Worker Context. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@n-db-ReadAllData
Patch Set: Created 5 years, 9 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
Index: content/browser/storage_partition_impl.cc
diff --git a/content/browser/storage_partition_impl.cc b/content/browser/storage_partition_impl.cc
index 55f3c73cd4b184f9dc7533fa1f9518c61c2da44d..29d434a61cbc632b5a7d89867a39bba220e7716c 100644
--- a/content/browser/storage_partition_impl.cc
+++ b/content/browser/storage_partition_impl.cc
@@ -417,6 +417,9 @@ StoragePartitionImpl::~StoragePartitionImpl() {
if (GetGeofencingManager())
GetGeofencingManager()->Shutdown();
+
+ if (GetPlatformNotificationContext())
+ GetPlatformNotificationContext()->Shutdown();
}
StoragePartitionImpl* StoragePartitionImpl::Create(
@@ -500,7 +503,7 @@ StoragePartitionImpl* StoragePartitionImpl::Create(
new NavigatorConnectServiceWorkerServiceFactory(service_worker_context)));
scoped_refptr<PlatformNotificationContextImpl> platform_notification_context =
- new PlatformNotificationContextImpl(path);
+ new PlatformNotificationContextImpl(path, service_worker_context);
StoragePartitionImpl* storage_partition = new StoragePartitionImpl(
context, partition_path, quota_manager.get(), appcache_service.get(),

Powered by Google App Engine
This is Rietveld 408576698