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

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 4ec2db715252edfee3d7abc2bd9e4989b36783bb..f98cb25c522f91d0ae58f293a2c3ed82909f96e0 100644
--- a/content/browser/storage_partition_impl.cc
+++ b/content/browser/storage_partition_impl.cc
@@ -422,6 +422,9 @@ StoragePartitionImpl::~StoragePartitionImpl() {
if (GetGeofencingManager())
GetGeofencingManager()->Shutdown();
+
+ if (GetPlatformNotificationContext())
+ GetPlatformNotificationContext()->Shutdown();
}
StoragePartitionImpl* StoragePartitionImpl::Create(
@@ -510,7 +513,8 @@ StoragePartitionImpl* StoragePartitionImpl::Create(
new NavigatorConnectServiceWorkerServiceFactory(service_worker_context)));
scoped_refptr<PlatformNotificationContextImpl> platform_notification_context =
- new PlatformNotificationContextImpl(path);
+ new PlatformNotificationContextImpl(path, service_worker_context);
+ platform_notification_context->Initialize();
StoragePartitionImpl* storage_partition = new StoragePartitionImpl(
context, partition_path, quota_manager.get(), appcache_service.get(),

Powered by Google App Engine
This is Rietveld 408576698