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

Unified Diff: content/renderer/service_worker/service_worker_context_client.cc

Issue 2404233002: Aborts all pending fetch event callbacks in ServiceWorkerContextClient::willDestroyWorkerContext. (Closed)
Patch Set: incorporated shimazu's comment Created 4 years, 2 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/stop-worker-with-pending-fetch.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/service_worker/service_worker_context_client.cc
diff --git a/content/renderer/service_worker/service_worker_context_client.cc b/content/renderer/service_worker/service_worker_context_client.cc
index 7773587bf3fec6423f7d85a8fe4574c37e2b0dcf..b321f8988fd1f08a3ace0cdb04338809cc4a4a7e 100644
--- a/content/renderer/service_worker/service_worker_context_client.cc
+++ b/content/renderer/service_worker/service_worker_context_client.cc
@@ -472,6 +472,12 @@ void ServiceWorkerContextClient::willDestroyWorkerContext(
it.GetCurrentValue()->Run(blink::mojom::ServiceWorkerEventStatus::ABORTED,
base::Time::Now());
}
+ // Aborts the all pending fetch event callbacks.
+ for (WorkerContextData::FetchEventCallbacksMap::iterator it(
+ &context_->fetch_event_callbacks);
+ !it.IsAtEnd(); it.Advance()) {
+ it.GetCurrentValue()->Run(SERVICE_WORKER_ERROR_ABORT, base::Time::Now());
+ }
// We have to clear callbacks now, as they need to be freed on the
// same thread.
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/stop-worker-with-pending-fetch.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698