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

Unified Diff: content/browser/renderer_host/render_process_host_impl.cc

Issue 2119973002: Port WebSockets to Mojo IPC (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index 4c2131cb6b8ca0fb514714c4232ff6964f84e936..99e2ebd68ce6d7416e4004d788c056070957896f 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -111,7 +111,6 @@
#include "content/browser/renderer_host/render_widget_helper.h"
#include "content/browser/renderer_host/render_widget_host_impl.h"
#include "content/browser/renderer_host/text_input_client_message_filter.h"
-#include "content/browser/renderer_host/websocket_dispatcher_host.h"
#include "content/browser/resolve_proxy_msg_helper.h"
#include "content/browser/service_worker/service_worker_context_wrapper.h"
#include "content/browser/service_worker/service_worker_dispatcher_host.h"
@@ -121,6 +120,7 @@
#include "content/browser/storage_partition_impl.h"
#include "content/browser/streams/stream_context.h"
#include "content/browser/tracing/trace_message_filter.h"
+#include "content/browser/websocket/websocket_manager.h"
#include "content/browser/webui/web_ui_controller_factory_registry.h"
#include "content/common/child_process_host_impl.h"
#include "content/common/child_process_messages.h"
@@ -951,14 +951,11 @@ void RenderProcessHostImpl::CreateMessageFilters() {
AddFilter(new BrowserCdmManager(GetID(), NULL));
#endif
- WebSocketDispatcherHost::GetRequestContextCallback
- websocket_request_context_callback(
- base::Bind(&GetRequestContext, request_context, media_request_context,
- RESOURCE_TYPE_SUB_RESOURCE));
-
- AddFilter(new WebSocketDispatcherHost(
- GetID(), websocket_request_context_callback, blob_storage_context.get(),
- storage_partition_impl_));
+ // TODO(darin): Find a better spot for this setup code.
+ websocket_manager_.reset(
Ben Goodger (Google) 2016/07/01 21:19:51 Is it possible to do this in response to a request
darin (slow to review) 2016/07/06 16:57:21 Done.
+ new WebSocketManager(GetID(),
+ storage_partition_impl_,
+ blob_storage_context));
message_port_message_filter_ = new MessagePortMessageFilter(
base::Bind(&RenderWidgetHelper::GetNextRoutingID,

Powered by Google App Engine
This is Rietveld 408576698