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

Unified Diff: content/browser/service_worker/service_worker_context_core.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 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_context_core.cc
diff --git a/content/browser/service_worker/service_worker_context_core.cc b/content/browser/service_worker/service_worker_context_core.cc
index 2272c95851bf8eec3c0f5a09dd63d3b4d2ab4455..b199b48a72bc75a44d66b89b95c1072e51d09358 100644
--- a/content/browser/service_worker/service_worker_context_core.cc
+++ b/content/browser/service_worker/service_worker_context_core.cc
@@ -401,6 +401,39 @@ void ServiceWorkerContextCore::SetBlobParametersForCache(
blob_storage_context);
}
+scoped_ptr<ServiceWorkerProviderHost>
+ServiceWorkerContextCore::TransferProviderHostOut(
+ int process_id, int provider_id) {
+ ProviderMap* map = GetProviderMapForProcess(process_id);
+ ServiceWorkerProviderHost* transferee = map->Lookup(provider_id);
+ int frame_id = transferee->frame_id();
+ transferee->PrepareForCrossSiteTransfer();
+
+ ServiceWorkerProviderHost* replacement =
+ new ServiceWorkerProviderHost(process_id,
+ frame_id,
+ provider_id,
+ AsWeakPtr(),
+ transferee->dispatcher_host());
michaeln 2014/12/10 00:39:27 ooops, i see a bug here, dispatcher_host() will al
+ map->Replace(provider_id, replacement);
+
+ return make_scoped_ptr(transferee);
+}
+
+void ServiceWorkerContextCore::TransferProviderHostIn(
+ int new_process_id, int new_provider_id,
+ scoped_ptr<ServiceWorkerProviderHost> transferree) {
+ ProviderMap* map = GetProviderMapForProcess(new_process_id);
+ ServiceWorkerProviderHost* temp = map->Lookup(new_provider_id);
+ DCHECK(temp->document_url().is_empty());
+ transferree->CompleteCrossSiteTransfer(new_process_id,
+ temp->frame_id(),
+ new_provider_id,
+ temp->dispatcher_host());
+ map->Replace(new_provider_id, transferree.release());
+ delete temp;
+}
+
void ServiceWorkerContextCore::OnWorkerStarted(ServiceWorkerVersion* version) {
if (!observer_list_.get())
return;

Powered by Google App Engine
This is Rietveld 408576698