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

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

Issue 189253002: Implement ServiceWorker::postMessage() [Chromium] (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rename message handler per review feedback Created 6 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 9462bcda3a543ac0d87aad21f0bfecfbc78576f4..3970f70901e1fbe2e6142632b266ec2c033f1758 100644
--- a/content/browser/service_worker/service_worker_dispatcher_host.cc
+++ b/content/browser/service_worker/service_worker_dispatcher_host.cc
@@ -5,10 +5,12 @@
#include "content/browser/service_worker/service_worker_dispatcher_host.h"
#include "base/strings/utf_string_conversions.h"
+#include "content/browser/message_port_message_filter.h"
+#include "content/browser/message_port_service.h"
#include "content/browser/service_worker/embedded_worker_registry.h"
#include "content/browser/service_worker/service_worker_context_core.h"
#include "content/browser/service_worker/service_worker_context_wrapper.h"
-#include "content/browser/service_worker/service_worker_provider_host.h"
+#include "content/browser/service_worker/service_worker_registration.h"
#include "content/browser/service_worker/service_worker_utils.h"
#include "content/common/service_worker/embedded_worker_messages.h"
#include "content/common/service_worker/service_worker_messages.h"
@@ -35,11 +37,12 @@ const uint32 kFilteredMessageClasses[] = {
namespace content {
ServiceWorkerDispatcherHost::ServiceWorkerDispatcherHost(
- int render_process_id)
- : BrowserMessageFilter(
- kFilteredMessageClasses, arraysize(kFilteredMessageClasses)),
- render_process_id_(render_process_id) {
-}
+ int render_process_id,
+ MessagePortMessageFilter* message_port_message_filter)
+ : BrowserMessageFilter(kFilteredMessageClasses,
+ arraysize(kFilteredMessageClasses)),
+ render_process_id_(render_process_id),
+ message_port_message_filter_(message_port_message_filter) {}
ServiceWorkerDispatcherHost::~ServiceWorkerDispatcherHost() {
if (context_) {
@@ -85,6 +88,7 @@ bool ServiceWorkerDispatcherHost::OnMessageReceived(
OnAddScriptClient)
IPC_MESSAGE_HANDLER(ServiceWorkerHostMsg_RemoveScriptClient,
OnRemoveScriptClient)
+ IPC_MESSAGE_HANDLER(ServiceWorkerHostMsg_PostMessage, OnPostMessage)
IPC_MESSAGE_HANDLER(EmbeddedWorkerHostMsg_WorkerStarted,
OnWorkerStarted)
IPC_MESSAGE_HANDLER(EmbeddedWorkerHostMsg_WorkerStopped,
@@ -158,6 +162,54 @@ void ServiceWorkerDispatcherHost::OnUnregisterServiceWorker(
request_id));
}
+void ServiceWorkerDispatcherHost::OnPostMessage(
+ int64 registration_id,
+ const base::string16& message,
+ const std::vector<int>& sent_message_port_ids) {
+ if (!context_ || !ServiceWorkerUtils::IsFeatureEnabled())
+ return;
+
+ std::vector<int> new_routing_ids(sent_message_port_ids.size());
+ for (size_t i = 0; i < sent_message_port_ids.size(); ++i) {
+ new_routing_ids[i] = message_port_message_filter_->GetNextRoutingID();
+ MessagePortService::GetInstance()->UpdateMessagePort(
+ sent_message_port_ids[i],
+ message_port_message_filter_,
+ new_routing_ids[i]);
+ }
+
+ context_->storage()->FindRegistrationForId(
+ registration_id,
+ base::Bind(&ServiceWorkerDispatcherHost::PostMessageFoundRegistration,
+ message,
+ sent_message_port_ids,
+ new_routing_ids));
+}
+
+namespace {
+void NoOpStatusCallback(ServiceWorkerStatusCode status) {}
+} // namespace
+
+// static
+void ServiceWorkerDispatcherHost::PostMessageFoundRegistration(
+ const base::string16& message,
+ const std::vector<int>& sent_message_port_ids,
+ const std::vector<int>& new_routing_ids,
+ ServiceWorkerStatusCode status,
+ const scoped_refptr<ServiceWorkerRegistration>& result) {
+ if (status != SERVICE_WORKER_OK)
+ return;
+ DCHECK(result);
+
+ // TODO(jsbell): Route message to appropriate version. crbug.com/351797
+ ServiceWorkerVersion* version = result->active_version();
+ if (!version)
+ return;
+ version->SendMessage(
+ ServiceWorkerMsg_Message(message, sent_message_port_ids, new_routing_ids),
+ base::Bind(&NoOpStatusCallback));
+}
+
void ServiceWorkerDispatcherHost::OnProviderCreated(int provider_id) {
if (!context_)
return;

Powered by Google App Engine
This is Rietveld 408576698