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

Unified Diff: content/renderer/websharedworker_proxy.cc

Issue 2601893002: SharedWorker: Remove message forwarding mechanism (Closed)
Patch Set: Created 4 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/renderer/websharedworker_proxy.cc
diff --git a/content/renderer/websharedworker_proxy.cc b/content/renderer/websharedworker_proxy.cc
index 880a6960762858c227ef8d7b65b1d89fcca3c193..3cf6f0cf8bf931fc4f6343516243d6289ed6ea4e 100644
--- a/content/renderer/websharedworker_proxy.cc
+++ b/content/renderer/websharedworker_proxy.cc
@@ -53,7 +53,7 @@ bool WebSharedWorkerProxy::Send(std::unique_ptr<IPC::Message> message) {
// For now we proxy all messages to the worker process through the browser.
// Revisit if we find this slow.
// TODO(jabdelmalek): handle sync messages if we need them.
- return router_->Send(new ViewHostMsg_ForwardToWorker(*message));
+ return router_->Send(message.release());
}
void WebSharedWorkerProxy::SendQueuedMessages() {
@@ -76,8 +76,8 @@ void WebSharedWorkerProxy::connect(blink::WebMessagePortChannel* channel,
DCHECK_NE(MSG_ROUTING_NONE, message_port_id);
webchannel->QueueMessages();
- Send(base::MakeUnique<WorkerMsg_Connect>(route_id_, message_port_id,
- MSG_ROUTING_NONE));
+ Send(base::MakeUnique<ViewHostMsg_ConnectToWorker>(route_id_,
+ message_port_id));
connect_listener_ = listener;
}
« content/browser/shared_worker/shared_worker_service_impl.cc ('K') | « content/common/view_messages.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698