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

Unified Diff: content/renderer/render_view_impl.cc

Issue 2483593002: RenderWidget: hoist synchronous IPC out of Init methods. (Closed)
Patch Set: Rework Created 4 years, 1 month 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
« no previous file with comments | « no previous file | content/renderer/render_widget.h » ('j') | content/renderer/render_widget.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index a20ef4d509906ad6416e2038fe64c3a89545c9dc..daaaaa72d94c85729ef7a6371ef90820be11553a 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -701,21 +701,20 @@ RenderViewImpl::RenderViewImpl(CompositorDependencies* compositor_deps,
void RenderViewImpl::Initialize(const mojom::CreateViewParams& params,
bool was_created_by_renderer) {
- SetRoutingID(params.view_id);
-
- int opener_view_routing_id;
+ int opener_view_routing_id = MSG_ROUTING_NONE;
WebFrame* opener_frame = RenderFrameImpl::ResolveOpener(
params.opener_frame_route_id, &opener_view_routing_id);
- if (opener_view_routing_id != MSG_ROUTING_NONE && was_created_by_renderer)
- opener_id_ = opener_view_routing_id;
+ if (!was_created_by_renderer)
+ opener_view_routing_id = MSG_ROUTING_NONE;
display_mode_ = params.initial_size.display_mode;
+ // TODO(nick): Make sure that WebView::create doesn't need routing_id_ set.
webview_ =
WebView::create(this, is_hidden() ? blink::WebPageVisibilityStateHidden
: blink::WebPageVisibilityStateVisible);
- RenderWidget::DoInit(MSG_ROUTING_NONE, webview_->widget(),
- CreateWidgetCallback());
+ RenderWidget::Init(params.view_id, opener_view_routing_id,
+ webview_->widget());
g_view_map.Get().insert(std::make_pair(webview(), this));
g_routing_id_view_map.Get().insert(std::make_pair(GetRoutingID(), this));
« no previous file with comments | « no previous file | content/renderer/render_widget.h » ('j') | content/renderer/render_widget.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698