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

Unified Diff: content/browser/service_worker/service_worker_provider_host.cc

Issue 702843004: Transfer serviceworker state during cross site navigations too. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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/service_worker/service_worker_provider_host.cc
diff --git a/content/browser/service_worker/service_worker_provider_host.cc b/content/browser/service_worker/service_worker_provider_host.cc
index e16454558e0ec9cc46862b3110a811dcf3bfca6f..2b1198050b309f7a5d32401b013e483d43301490 100644
--- a/content/browser/service_worker/service_worker_provider_host.cc
+++ b/content/browser/service_worker/service_worker_provider_host.cc
@@ -214,6 +214,23 @@ void ServiceWorkerProviderHost::AddScopedProcessReferenceToPattern(
IncreaseProcessReference(pattern);
}
+void ServiceWorkerProviderHost::PrepareForCrossSiteTransfer() {
+ DCHECK(!associated_registration());
+ process_id_ = 0;
falken 2014/11/11 08:38:57 EmbeddedWorkerInstance uses -1 for "invalid proces
michaeln 2014/11/13 00:05:08 Done
+ provider_id_ = kInvalidServiceWorkerProviderId;
+ dispatcher_host_ = nullptr;
+}
+
+void ServiceWorkerProviderHost::CompleteCrossSiteTransfer(
+ int new_process_id,
+ int new_provider_id,
+ ServiceWorkerDispatcherHost* new_dispatcher_host) {
+ DCHECK(!dispatcher_host_);
+ process_id_ = new_process_id;
+ provider_id_ = new_provider_id;
+ dispatcher_host_ = new_dispatcher_host;
+}
+
ServiceWorkerObjectInfo ServiceWorkerProviderHost::CreateHandleAndPass(
ServiceWorkerVersion* version) {
ServiceWorkerObjectInfo info;

Powered by Google App Engine
This is Rietveld 408576698