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

Unified Diff: content/child/service_worker/service_worker_network_provider.cc

Issue 2023733002: service worker: Remove unused PROVIDER_FOR_SANDBOXED_IFRAME (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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/child/service_worker/service_worker_network_provider.cc
diff --git a/content/child/service_worker/service_worker_network_provider.cc b/content/child/service_worker/service_worker_network_provider.cc
index dab3db7517595f4598357183fb9f3af54ad28a42..0632e81e09b3cbaa83e0fbb0e0e0267c160bd782 100644
--- a/content/child/service_worker/service_worker_network_provider.cc
+++ b/content/child/service_worker/service_worker_network_provider.cc
@@ -24,15 +24,6 @@ int GetNextProviderId() {
return sequence.GetNext(); // We start at zero.
}
-// When the provider is for a sandboxed iframe we use
-// kInvalidServiceWorkerProviderId as the provider type and we don't create
-// ServiceWorkerProviderContext and ServiceWorkerProviderHost.
-int GenerateProviderIdForType(const ServiceWorkerProviderType provider_type) {
- if (provider_type == SERVICE_WORKER_PROVIDER_FOR_SANDBOXED_FRAME)
- return kInvalidServiceWorkerProviderId;
- return GetNextProviderId();
-}
-
} // namespace
void ServiceWorkerNetworkProvider::AttachToDocumentState(
@@ -121,7 +112,7 @@ ServiceWorkerNetworkProvider::ServiceWorkerNetworkProvider(
ServiceWorkerProviderType provider_type)
: ServiceWorkerNetworkProvider(route_id,
provider_type,
- GenerateProviderIdForType(provider_type)) {}
+ GetNextProviderId()) {}
ServiceWorkerNetworkProvider::ServiceWorkerNetworkProvider()
: provider_id_(kInvalidServiceWorkerProviderId) {}
« no previous file with comments | « content/browser/service_worker/service_worker_provider_host.cc ('k') | content/common/service_worker/service_worker_types.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698