Commit 7a36c415 authored by Han Leon's avatar Han Leon Committed by Commit Bot

[ServiceWorker] A shared worker has no means to call ServiceWorker#postMessage

BUG=

Change-Id: Iaf37fa21c8b7c05996db17f33778bc5a4b0271c0
Reviewed-on: https://chromium-review.googlesource.com/958233
Commit-Queue: Han Leon <leon.han@intel.com>
Reviewed-by: default avatarMatt Falkenhagen <falken@chromium.org>
Reviewed-by: default avatarKinuko Yasuda <kinuko@chromium.org>
Reviewed-by: default avatarMakoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542434}
parent 95253d13
......@@ -249,9 +249,6 @@ void ServiceWorkerHandle::DispatchExtendableMessageEvent(
}
switch (provider_host_->provider_type()) {
case blink::mojom::ServiceWorkerProviderType::kForWindow:
// TODO(leonhsl): Move kForSharedWorker to the kUnknown block to clarify
// that currently a shared worker can not postMessage to a service worker.
case blink::mojom::ServiceWorkerProviderType::kForSharedWorker:
service_worker_client_utils::GetClient(
provider_host_.get(),
base::BindOnce(&DispatchExtendableMessageEventFromClient, version_,
......@@ -272,6 +269,9 @@ void ServiceWorkerHandle::DispatchExtendableMessageEvent(
provider_host_));
return;
}
case blink::mojom::ServiceWorkerProviderType::kForSharedWorker:
// Shared workers don't yet have access to ServiceWorker objects, so they
// can't postMessage to one (https://crbug.com/371690).
case blink::mojom::ServiceWorkerProviderType::kUnknown:
break;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment