Make sure Widget mojo interface is constructed for RenderWidgetHosts.
We require that a Widget interface exists for a RenderWidgetHost. Ensure
that it is plumbed correctly so that in has a bound request interface
always.
BUG=722928
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Change-Id: Ibc786aca96b2ae332e4d7ff54ecd07f10ce143db
Reviewed-on: https://chromium-review.googlesource.com/598121
Commit-Queue: Dave Tapuska <[email protected]>
Reviewed-by: Daniel Cheng <[email protected]>
Reviewed-by: Antoine Labour <[email protected]>
Cr-Commit-Position: refs/heads/master@{#492475}diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 67cffc5..0eed782 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1713,10 +1713,9 @@
site_instance->GetProcess()->GetNextRoutingID();
}
- GetRenderManager()->Init(site_instance.get(), view_routing_id,
- params.main_frame_routing_id,
- main_frame_widget_routing_id,
- params.renderer_initiated_creation);
+ GetRenderManager()->Init(
+ site_instance.get(), view_routing_id, params.main_frame_routing_id,
+ main_frame_widget_routing_id, params.renderer_initiated_creation);
// blink::FrameTree::setName always keeps |unique_name| empty in case of a
// main frame - let's do the same thing here.
@@ -2421,8 +2420,8 @@
return;
}
- RenderWidgetHostImpl* widget_host =
- new RenderWidgetHostImpl(this, process, route_id, IsHidden());
+ RenderWidgetHostImpl* widget_host = new RenderWidgetHostImpl(
+ this, process, route_id, std::move(widget), IsHidden());
RenderWidgetHostViewBase* widget_view =
static_cast<RenderWidgetHostViewBase*>(
