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

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

Issue 1701843002: ServiceWorker: Implement 'source' and 'origin' attributes of ExtendableMessageEvent (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@move_focus_into_utils
Patch Set: clean up tests Created 4 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/service_worker/service_worker_dispatcher_host.cc
diff --git a/content/browser/service_worker/service_worker_dispatcher_host.cc b/content/browser/service_worker/service_worker_dispatcher_host.cc
index 7a6c4828bcfe05cac62038930f6545d5198dafe7..6f7568170abf71d0ecebd08b4768148e587b6cdd 100644
--- a/content/browser/service_worker/service_worker_dispatcher_host.cc
+++ b/content/browser/service_worker/service_worker_dispatcher_host.cc
@@ -676,7 +676,9 @@ void ServiceWorkerDispatcherHost::OnGetRegistrationForReady(
void ServiceWorkerDispatcherHost::OnPostMessageToWorker(
int handle_id,
+ int provider_id,
const base::string16& message,
+ const url::Origin& source_origin,
const std::vector<TransferredMessagePort>& sent_message_ports) {
TRACE_EVENT0("ServiceWorker",
"ServiceWorkerDispatcherHost::OnPostMessageToWorker");
@@ -689,8 +691,15 @@ void ServiceWorkerDispatcherHost::OnPostMessageToWorker(
return;
}
+ ServiceWorkerProviderHost* sender_provider_host =
+ GetContext()->GetProviderHost(render_process_id_, provider_id);
+ if (!sender_provider_host) {
+ bad_message::ReceivedBadMessage(this, bad_message::SWDH_POST_MESSAGE);
+ return;
+ }
+
handle->version()->DispatchExtendableMessageEvent(
- message, sent_message_ports,
+ sender_provider_host, message, source_origin, sent_message_ports,
base::Bind(&ServiceWorkerUtils::NoOpStatusCallback));
}

Powered by Google App Engine
This is Rietveld 408576698